Hotfix 3.1.8 #35
Merged
Hotfix 3.1.8 #35
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Switch header buttons to use HTML button element for better UX * Adjust styling to restoring honoring header button text color * Improve header button layout on <768px display
Setting transparent background for button-tertiary class.
* Minor editor fix - Remove head from editor due to causing more disruptions than benefits due to variations within portals central css stylings. Best to integrate head into new redesign of occurrence editor. * Update hotfix version (3.1.6) * closes #1954 Fixes use of hidden button and button submit value Fix transfer taxa form so that action is submited in the request * Fixing Login Form CSS closes #1975 # Issue #1975 # Summary input id value for email was "login" and css rule `#login {float: right}` was being applied to this accidently causing some odd behavior. Changed id of login so the collid wouldn't happen and bumped the width of the form so that it will display correctly. * resolve merge conflict * Occurrence Profile bugs - Adjust editor permissions check so that it includes creator of general research collections (observerUid = active user) - Avoid double sanitation of identifier and collector * Update geolocate.php (#1996) Fixes minor typo that occurs in error message * hotfix - protection - If collid input is a number + single quote, assume it's an SQL Inject support and set value to 0, which returns nothing, rather than putting a load on the server * Institution Sanitation issue - Remove sanitation off outbound notes content to avoid interfering with embedded html tags that are added when this field is appended with GriSciColl info. - Remove sanitation notes which were only meant to communicate to internal team when sanitation content was originally added. Resolves following issue, in part: #1982 * API Annotation Bug - Fix issue with missing recordID field from SQL statement definition * added the changes to hotfix * Bug adding image - If user is null, user verification code incorrectly checks to see if there is a user with an empty string username or email. Thus, add code that skips checking user table if login details are an empty value - Don't add empty strings to database. Keep them as null values. - Comment out user verification check. Just test to make sure it's a number. * remove associations changes * Closes #2040 Sorts By Sciname within family (#2052) # Issue #2040 # Summary Adds extra sort conditions so that records are sorted by sciname after being storted by family. * Closes #2049 Fixes typo on globals variable # Issues #2049 # Summary Fixes typo for `IMAGE_ROOT_PATH` and `IMAGE_ROOT_URL` global variables. Note this will be overritten in the coming 3.2 changes with the multimedia changes so maybe it would be worth merging. * Fix country synonyms, some U.S. states, add U.S. state abbreviations (#2059) * Closes #2064 Fixing String Number multiply # Issues #2064 # Summary Adds as is_numeric check on 'page' request variable so that the `$pageNumber` variable is alwasys a number * Update geolocate.php per Nelson's suggestion (#2066) See #1702 (comment) Co-authored-by: Edward Gilbert <[email protected]> * Update db_schema_patch-3.1.sql - Explicitly set the index for omoccurrences.locality to a length of 100, thus avoid DB setting it to a default larger length that is beyond what is needed nor practical. Addresses issue: #2050 * removed arrow functions and union types * replaced str_contains with str_pos --------- Co-authored-by: Edward Gilbert <[email protected]> Co-authored-by: MuchQuak <[email protected]> Co-authored-by: Logan Wilt <[email protected]> Co-authored-by: atticus29 <[email protected]> Co-authored-by: Lindsay Walker <[email protected]> Co-authored-by: Nikita Salikov <[email protected]> Co-authored-by: NikitaSalikov <[email protected]> Co-authored-by: Katie Pearson <[email protected]>
Hotfix version 3.1.6.1 * Update index.php Build occurrence array after protections applied * Adjustment - Get observeruid value directly from occData class array, thus avoiding occArr needing to exist prior to permission checks --------- Co-authored-by: Edward Gilbert <[email protected]> Co-authored-by: Katie Pearson <[email protected]>
Revert code version to 3.2
This reverts commit 37e7d1a.
* Fix coll profile slowdown (#2092) * Update collprofiles.php Remove Bionomia badge curl request Use onerror to hide Bionomia badge if not loading. * Update symbbase.php Update code version to 3.1.7 * Update collprofiles.php Fix for missing variable. * Improvement SQL efficiency - Improve efficiency of 2 very commonly used SQL queries --------- Co-authored-by: Greg Post <[email protected]>
**3.1.8 Hotfix update** - API developments -- Fix bug associated with displaying all media. Previous output was limited to first 200 records only -- Add limit and offset variables to display all media -- Add filters to display all media endpoint: tid filter, includeSynonyms, includeChildren - Out of bounds hyperlink within occurrence editor. Resolves #2099 Co-authored-by: Greg Post <[email protected]> Co-authored-by: Mark <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.