Skip to content

[Raisinbread] Update Raisinbread SQL files and remove unnecessary MariaDB sandbox comments#10512

Open
HachemJ wants to merge 5 commits into
aces:mainfrom
HachemJ:FixUntrackedRaisinbreadSQLFiles
Open

[Raisinbread] Update Raisinbread SQL files and remove unnecessary MariaDB sandbox comments#10512
HachemJ wants to merge 5 commits into
aces:mainfrom
HachemJ:FixUntrackedRaisinbreadSQLFiles

Conversation

@HachemJ
Copy link
Copy Markdown

@HachemJ HachemJ commented May 27, 2026

Brief summary of changes

This PR updates the DB_dump_table_data php script to remove unnecessary MariaDB sandbox comment /*M!999999\- enable the sandbox mode */ from exported SQL files.
It also adds missing RB files to the raisinbread directory, and picks up some existing raisinbread changes that were present in the database but had not yet been committed through previous PRs.

Note: Most RB_file diffs are formatting only changes introduced during export regeneration

HachemJ added 3 commits May 27, 2026 12:44
…ment /*M!999999\- enable the sandbox mode */
…dump_table_data.php script (only exception is I have replaced my host IP by localhost)
@github-actions github-actions Bot added Language: SQL PR or issue that update SQL code RaisinBread PR or issue introducing/requiring improvements to the Raidinbread dataset Language: PHP PR or issue that update PHP code labels May 27, 2026
@HachemJ HachemJ changed the title [Raisinbread] Update Raisinbread SQL files and remove unnecessary [Raisinbread] Update Raisinbread SQL files and remove unnecessary MariaDB sandbox comments May 27, 2026
@skarya22
Copy link
Copy Markdown
Contributor

skarya22 commented May 28, 2026

@ridz1208 after merging this, can you look at mine #10515

@skarya22 skarya22 self-assigned this Jun 2, 2026
@skarya22 skarya22 self-requested a review June 2, 2026 17:46
Copy link
Copy Markdown
Contributor

@skarya22 skarya22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks Jad

@skarya22 skarya22 added the Passed manual tests PR has been successfully tested by at least one peer label Jun 3, 2026
@@ -1,6 +1,6 @@
SET FOREIGN_KEY_CHECKS=0;
TRUNCATE TABLE `users`;
LOCK TABLES `users` WRITE;
INSERT INTO `users` (`ID`, `UserID`, `Password`, `Real_name`, `First_name`, `Last_name`, `Degree`, `Position_title`, `Institution`, `Department`, `Address`, `City`, `State`, `Zip_code`, `Country`, `Phone`, `Fax`, `Email`, `Privilege`, `PSCPI`, `DBAccess`, `Active`, `Password_hash`, `PasswordChangeRequired`, `Pending_approval`, `Doc_Repo_Notifications`, `language_preference`, `active_from`, `active_to`, `account_request_date`) VALUES (1,'admin',NULL,'Admin account','Admin','account',NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,'admin@example.com',0,'N','','Y','$2y$10$6pjQ.x5rPY7voNFs2w/eI.pzLL8H/9wPZ98nabvBOEwDBClhK0l1S',0,'N','N',NULL,NULL,NULL,NULL);
INSERT INTO `users` (`ID`, `UserID`, `Password`, `Real_name`, `First_name`, `Last_name`, `Degree`, `Position_title`, `Institution`, `Department`, `Address`, `City`, `State`, `Zip_code`, `Country`, `Phone`, `Fax`, `Email`, `Privilege`, `PSCPI`, `DBAccess`, `Active`, `Password_hash`, `PasswordChangeRequired`, `TOTPSecret`, `Pending_approval`, `Doc_Repo_Notifications`, `language_preference`, `active_from`, `active_to`, `account_request_date`) VALUES (1,'admin',NULL,'Admin account','Admin','account',NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL,'admin@example.com',0,'N','','Y','$2y$12$Kp6FRQNupQZuPiNWfofdDuxlEPNOLyCf55g0VCRrzJlN2bVEj3Bli',0,NULL,'N','N',NULL,NULL,NULL,NULL);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@HachemJ looks like you might have changed the password on RB when you submitted this, can you exclude this file so the password stays the same ?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually you cant get rid of the file because of the TOTPSerect addition
you just need to undo the password hash change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language: PHP PR or issue that update PHP code Language: SQL PR or issue that update SQL code Passed manual tests PR has been successfully tested by at least one peer RaisinBread PR or issue introducing/requiring improvements to the Raidinbread dataset

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants