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.
To minimise open issues, I suggest adding dependencies to the main module and crawler so that we can confirm that the users have the correct modules installed. Patch to be provided shortly.
Comment | File | Size | Author |
---|---|---|---|
#5 | boost-7.x-1.x-dev-listmodule-1806158.patch | 5.73 KB | Anonymous (not verified) |
#1 | boost-1806158-expires-dependency.patch | 636 bytes | Anonymous (not verified) |
Comments
Comment #1
Anonymous (not verified) CreditAttribution: Anonymous commentedPatch for boost.info recommends expire module, adds Cache Expiration as a dependency for the boost_crawler.info. The "recommends" is debatable for boost.info since on admin/reports/status
boost is marked up as implementing the hook, also as it's a request to minimise issues and check configuration it may be worth making it a dependency even if a site administrator has boost set to expire the pages rarely (or not at all), there's probably even a case for remerging expire back into boost.
Comment #2
bgm CreditAttribution: bgm commentedThanks for the patch. Committed to 7.x-1.x: a0b4fc503ca1618b24c6822f02437b4e37a3c0a9
The "Cache Expiration" message in admin/reports/status is shown by the "expire" module, so I do not see any contradiction.
Might be interesting to show the status for a "recommended boost configuration" in admin/settings/performance/boost so that people could see at a glance which features are on/off, why they should install "expire", crawler/httprl, globalredirect, etc. Thoughts?
Comment #3
Anonymous (not verified) CreditAttribution: Anonymous commentedI'd be prepared to take that on if you'll push through the patch on :-)
http://drupal.org/node/1440754#comment-6574080
it's a change to the default Options +FollowSymLinks to Options +SymLinksIfOwnerMatch, adds a menu item in .htaccess generation and closes the 500 error page issue some people have with there ISP's.
Comment #4
Anonymous (not verified) CreditAttribution: Anonymous commentedLinking in #1826832: Optionally cache 404s for the new recommended modules page.
Comment #5
Anonymous (not verified) CreditAttribution: Anonymous commentedAttached is the patch as discussed above, a "boost modules" new tab an page that lists installation and description of relevant links to the project pages.
Modules listed
Also lists if a relevant module is installed but disabled, I believe the descriptions may be helpful to reduce support requests particularly boost crawler. (run through code review and corrected a couple of other little bits). New tabbed page takes the status report formatting for the table, if I've missed any relevant modules please inform me.
Comment #6
bgm CreditAttribution: bgm commentedCommitted to 7.x-1.x. Thanks for the patch!
I did some minor aesthetic changes:
* renamed tab "boost modules" to "related modules" (some modules are not boost-specific)
* small changes to the module descriptions.
* as in Drupal's "update status", I put the link to the project page on the module title, not a separate link. Also, I think it's good to have the link always there, in case people want to check the docs.