Generated with the help of an LLM.

Problem/Motivation

In 4.1.9, htmlawed.post_update.php starts with a docblock instead of <?php. When Drupal loads post-update files, PHP prints the file's source as plain
text:

  • /admin/reports/status shows the raw source above the page.
  • drush updatedb and drush deploy print the source.
  • htmlawed_post_update_announce_changed_default_config() is never defined, so the post-update never runs.

The 4.1.x branch has the same problem.

Steps to reproduce

  1. Update from 4.1.8 to 4.1.9.
  2. Visit /admin/reports/status or run drush updatedb.
  3. The source of htmlawed.post_update.php is printed.

Proposed resolution

Add the opening <?php tag to htmlawed.post_update.php. Patch attached.

Remaining tasks

Review and commit.

User interface changes

None.

API changes

None.

Data model changes

CommentFileSizeAuthor
htmlawed-post-update-missing-php-tag.patch291 bytesssawafta

Comments

ssawafta created an issue. See original summary.

  • 46ecaa6e committed on 4.1.x 
    Fixes #3634604 by @ssawafta: missing opening php tag in post-update hook...
alpha2zee’s picture

alpha2zee commented
Assigned: Unassigned » alpha2zee
Status: Needs review » Fixed

Thank you for informing of this silly mistake. Now fixed in new version 4.1.10.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

alpha2zee’s picture

alpha2zee commented
Status: Fixed » Closed (fixed)