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

500 error when trying to delete a determination from a specimen record #825

Closed
themerekat opened this issue Jan 2, 2024 · 3 comments
Closed
Assignees
Labels

Comments

@themerekat
Copy link
Collaborator

https://ecdysis.org/collections/editor/occurrenceeditor.php
image

@themerekat
Copy link
Collaborator Author

Then, when I go back to the Determinations tab, both determinations are listed as accepted, rather than one of the determinations being deleted (I tried to delete the "Testa" record):
image

@egbot
Copy link
Member

egbot commented Jan 4, 2024

I reopened this issue because suppressing the MySQL error is only meant as a temporary fix to ensure the page doesn't crash. The error in the code likely still remains. Currently working on fixing the root issue.

@egbot egbot reopened this Jan 4, 2024
@github-project-automation github-project-automation bot moved this from Done to In Progress in Symbiota issue triage Jan 4, 2024
@egbot egbot self-assigned this Jan 4, 2024
egbot added a commit that referenced this issue Jan 5, 2024
-Occurrence Editor
-- Add try-catch blocks for SQL insert statements that produce a fatal exception within PHP >v8.1 (#824, #825)
-- Avoid >8.1+ deprecated warnings associated with passing null variables to string functions
egbot added a commit that referenced this issue Jan 15, 2024
* Hotfix 2023-12-28
- Checklist and Identification key Management 
-- Identification Key: remove unnecessary input variable sanitation that was causing issues with display of characters within key
-- Remove connection sharing to avoid read-only connections being shared with ChecklistVoucherAdmin needing a writable connection
- OccurrenceAccessStats: Ensure that all SQL statements are set within a condition that checks that STORE_STATISTICS is set and true
- Occurrence Listing and Editor functions
-- Add try-catch blocks for SQL insert statements that produce a fatal exception within PHP >v8.1 (#824, #825)
-- Avoid >8.1+ deprecated warnings associated with passing null variables to string functions
-- Bug interfering with Coordinate download option listed within map tab of occurrence listing page
- Taxonomy and Taxon Profile Editor
-- Adjust TinyMCE description editor to not convert reference link URLs to relative paths (#794)
-- Fix SQL errors within taxonomy editor and loader that are now producing permanent error since 8.2 update (#823, #826)
- General code adjustments 
-- Replace deprecated use of FILTER_SANITIZE_STRING option within filter_var function
-- Sanitize integer input using filter_var and the FILTER_SANITIZE_NUMBER_INT option
-- Ensure that NULLs and undefined values are not evaluated by PHP string functions
-- Remove deprecated auto_detect_line_endings
-- Avoid >8.1 SQL fatal errors
-- Update UTF-8 and ISO-8859-1 encoding to remove deprecated utf8_decode and utf8_encode
-- Block null values being used within string functions (now deprecated)
@egbot
Copy link
Member

egbot commented Jan 16, 2024

Issue resolved: 1aa64b4

@egbot egbot closed this as completed Jan 16, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Symbiota issue triage Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

3 participants