Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SirsiDynix Issues: Not all dates are formatted based on setting in directory.php #320

Merged
merged 3 commits into from
Mar 28, 2018

Conversation

queryluke
Copy link
Contributor

1. Management Module

default dates should be formatted to the format set in directory.php

Review Procedure

  1. Edit managment/directory.php, line 73 to a different date format (d-m-Y)
  2. Edit management/js/common.js, line 39 to a matching date format (dd-mm-yyyy)
  3. Add new document, default date should be displayed in the new format
  4. On changing the date, it should be displayed in the new format
  5. Document should save without error
  6. Click on newly created document. Last document revision date should be displayed
    in the new format
  7. Edit the document, date should be in new format
  8. Change date, should be in new format
  9. should save without error

2. Resources Module, My Queue

  1. Dates in the My Queue > Saved Requests & Submitted Requests should be formatted to the format
    set in directory.php

Review Procedure

  1. Edit resources/directory.php, line 112 to a different date format (d-m-Y)
  2. Go to "My Queue"
    • If there are no saved or submitted requests, add one of each
  3. The Date Created for both the Saved and Submitted requests should both
    display in the new format.

3. Resources Module, Acquisition Form

Remove the input placeholder on the acquisition form. It is hardcoded with
"mm/dd/yyyy", which cannot be adjusted until date format is moved to a configuration file.
Removing it completely follows the convention of all other date input fields. I.e., no others have a placeholder.

Review procedure

  1. Create a new resource
  2. Click on the new resource
  3. Select "Acquisitions" from the left nav menu
  4. Click "Edit Order Information"
  5. "Sub Start" & "Current Sub End" should not have a placeholder

@PaulPoulain PaulPoulain added the bug This is a bug (not an enhancement) label Dec 12, 2017
@veggiematts
Copy link
Contributor

Could you resolve the conflicts, please?

…ing-issues

fixes merge conflicts with development
@queryluke
Copy link
Contributor Author

@veggiematts done. Thanks for the ping.

@cjc66 cjc66 added the important Not critical but should be addressed label Mar 22, 2018
@cjc66 cjc66 added this to the Version 3.0.0 Beta milestone Mar 22, 2018
@veggiematts
Copy link
Contributor

Test plan is now a bit different: the date format is in common/common_directory.php line 80.
Tested ok. Code's ok.
Please note that "Sub Start" and "Sub End" in Cost Informations still have placeholders when enhancedCostHistory is enabled.
@queryluke do you want to fix this in this PR or later in a separate one?

@queryluke
Copy link
Contributor Author

@veggiematts Thanks, I've removed those placeholders.

@veggiematts veggiematts merged commit b2058f9 into coral-erm:development Mar 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is a bug (not an enhancement) important Not critical but should be addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants