EBMS Tickets

Issue Number 813
Summary Drupal 10.2.2 breaks file uploads
Created 2024-02-12 08:06:52
Issue Type Bug
Submitted By Kline, Bob (NIH/NCI) [C]
Assigned To Kline, Bob (NIH/NCI) [C]
Status Closed
Resolved 2024-02-14 18:15:29
Resolution Fixed
Path /home/bkline/backups/jira/oceebms/issue.391238
Description

I have filed a bug report with the Drupal core project.

I'm trying to come up with a workaround.

Comment entered 2024-02-14 18:15:29 by Kline, Bob (NIH/NCI) [C]

The Drupal core team has implemented and tested a fix for the bug I reported which broke file uploads. However, I'm not confident that this fix will be incorporated into a Drupal release in time for Harpers Ferry., since we're hoping to accelerate the timetable for the EBMS release so that it coincides with the CentOS server replacements. Therefore I have modified the eleven places in the code where file_save_upload is invoked in the following nine source code files to avoid tripping on the Drupal bug:

  • web/modules/custom/ebms_article/src/Form/FullTextForm.php

  • web/modules/custom/ebms_article/src/Form/FullTextQueue.php

  • web/modules/custom/ebms_core/src/Form/PubtypeAncestors.php

  • web/modules/custom/ebms_doc/src/Form/DocForm.php

  • web/modules/custom/ebms_import/src/Form/ImportForm.php

  • web/modules/custom/ebms_import/src/Form/ImportInternalArticles.php

  • web/modules/custom/ebms_meeting/src/Form/MeetingForm.php

  • web/modules/custom/ebms_review/src/Form/ReviewerDocForm.php

  • web/modules/custom/ebms_summary/src/Form/SummaryMemberDocForm.php

I think the names of these files are descriptive enough that they should provide sufficient identification of which forms will need to be tested.

you should be able to import articles on EBMS DEV again.

https://github.com/NCIOCPL/ebms/commit/dd7e739

Comment entered 2024-03-19 11:42:44 by Boggess, Cynthia (NIH/NCI) [C]

Bonnie and I have tested importing and uploading files on QA in a variety of places and did not have any issues.

Comment entered 2024-04-18 12:45:17 by Osei-Poku, William (NIH/NCI) [C]

Verified on PROD. Thanks!

Elapsed: 0:00:00.000597