Skip to content

Refactor UI elements and improve password handling logic#630

Merged
fho4abcd merged 8 commits intoABCD-DEVCOM:masterfrom
rogercgui:some-improvements
Apr 5, 2026
Merged

Refactor UI elements and improve password handling logic#630
fho4abcd merged 8 commits intoABCD-DEVCOM:masterfrom
rogercgui:some-improvements

Conversation

@rogercgui
Copy link
Copy Markdown
Member

No description provided.

Update www/htdocs/central/dataentry/change_password.php JavaScript: remove deprecated language attribute from the <script> tag, consolidate PHP-built alert text into $msg_alerta and escape it with addslashes to prevent JS syntax errors from quotes in translations, declare confirm_password with var and add missing semicolons, and apply minor whitespace/formatting cleanup.
Normalize indentation, line breaks, and stray control characters in www/htdocs/central/dataentry/js/password_check.js. This commit only adjusts formatting for readability and maintainability; there are no functional changes to the password validation logic.
Add a guard in ReservesAssign so it returns an 8-element array of empty strings when no reservations are assigned, preventing 'Undefined variable' errors in calling code. Also update the CHANGE LOG entry formatting and add a comment explaining the new non-NULL return behavior; remove an extraneous separator line and make minor formatting adjustments.

Corrects old errors:

[05-Apr-2026 04:26:10 Europe/Berlin] PHP Warning: Trying to access array offset on value of type null in C:\xampp\htdocs\ABCD2\www\htdocs\central\circulation\devolver_ex.php on line 284

[05-Apr-2026 04:26:10 Europe/Berlin] PHP Warning: Trying to access array offset on value of type null in C:\xampp\htdocs\ABCD2\www\htdocs\central\circulation\devolver_ex.php on line 283
Normalize/fix character encoding in Portuguese language files to correct garbled accented characters. Updates applied to pt dbadmin.tab files for Linux/Windows and central language file, plus settings/admin_logs.php so UI strings render correctly and accented characters display as intended.
@fho4abcd fho4abcd merged commit 624385e into ABCD-DEVCOM:master Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants