Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
During the install phase (install.core.inc), help references to INSTALL.txt use a link – <a href="@install_txt">INSTALL.txt</a>
– while the status report in system.install does not. This patch makes system.install consistent with the installer. It's helpful from a usability standpoint since INSTALL.txt doesn't exist in the root directory, and the link takes you directly to the file.
Comment | File | Size | Author |
---|---|---|---|
#1 | install-txt-link-1981912-1.patch | 1.49 KB | kostajh |
Comments
Comment #1
kostajh CreditAttribution: kostajh commentedComment #2
webchickMoving to documentation component, since it seems more related to that than system.module.
Comment #3
jhodgdonCan someone manually test and verify that this link works -- if you click it, does the INSTALL.txt file appear in your browser window? If so, it seems like a good idea.
Comment #4
kostajh CreditAttribution: kostajh commentedFor what it's worth, I tested this manually.
For someone else who would like to:
chmod go-w /path/to/drupal8/sites/default
chown -R root /path/to/drupal8/sites/default
and make sure thatsites/default/settings.php
doesn't exist.Comment #5
jhodgdonI'm glad you have tested it! Since this involves a code and not just documentation change, I am not going to commit it until someone else other than the person who made the patch has done the same test (including clicking the link to verify the INSTALL.txt file is shown) and marked this issue "reviewed and tested by the community".
I'm marking this "Novice" in order to (hopefully) attract someone who can do this review, which should be fairly simple to do.
Comment #6
jhodgdonAnd moving it back into system.module in case that helps people know "it really does need a review". :)
Comment #7
theMusician CreditAttribution: theMusician commentedI tried this on a vanilla 8.x install built from the most recent pull and it worked just fine. The install.txt file was viewable via a link on the verify requirements page.
Comment #8
Anonymous (not verified) CreditAttribution: Anonymous commentedI tried this on a vanilla 8.x install - the link displayed on the Verify Requirements page and successfully displayed the install.txt file when accessed.
Comment #9
jhodgdonThanks to both of you for testing! I'll get it committed shortly unless one of the other core committers wants to do it.
Comment #10
theMusician CreditAttribution: theMusician commentedGlad to help. At our local meetup I was running a ladder session on reviewing issues and testing patches so it is fun to see a quick turnaround on an issue.
Comment #11
jhodgdonThanks again! Committed to 8.x. As this is a UI change and not critical or a bug/error/typo, we will not be backporting it to 7.x.