Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.admin/settings/releasemonitor --> The latest status has been retrieved for all active modules. See Releases.
Releases points to admin/releases which in (at least 5.x) should be admin/build/releases
Sorry for the rough report, I'm sickin tired atm.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | releases_page_path.patch | 1.46 KB | ewhipple |
Comments
Comment #1
ewhipple commentedOn admin/settings/releasemonitor, there is a link to admin/releases. For Drupal 5, this should be admin/build/releases.
The problem arises from the fact that the string "admin/releases" appears in two places in releasemonitor.module, but was only changed in one.
The attached patch moves the version-checking logic into a private function, _releases_page_path(), and substitutes a function call for the two places where the string was used.
Comment #2
ewhipple commentedForgot to change the status.
Comment #3
douggreen commentedThanks. Your patch was almost right - I also had to move the $version variable to the function.
As soon as the project is more stable, I plan on branching for 4.7, and problems (and code) like this will disappear.
Comment #4
ewhipple commentedThanks, Doug!