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.
Loading the admin UI produces multiple of said warnings.
Comment | File | Size | Author |
---|---|---|---|
#1 | skinr-variables-only-reference-1653564-1.patch | 638 bytes | stewart.adam |
Comments
Comment #1
stewart.adam CreditAttribution: stewart.adam commentedAttached patch resolves issue.
Comment #2
rogical CreditAttribution: rogical commentedIt works!
Comment #3
stewart.adam CreditAttribution: stewart.adam commentedSetting RTBC :)
Comment #4
artbussy CreditAttribution: artbussy commentedThis throws a new message on my local install: Notice: Undefined variable: title in skinr_ui_admin_skins() (line 458 of C:\xampp\htdocs\exampel\sites\all\modules\skinr\skinr_ui.admin.inc).
Comment #5
moonray CreditAttribution: moonray commentedI can't reproduce this issue @stewart.adam. What version of PHP are you using?
Comment #6
stewart.adam CreditAttribution: stewart.adam commentedPHP 5.2.17 (CentOS 5)
Comment #7
moonray CreditAttribution: moonray commentedSee http://drupal.org/requirements . Drupal 7 requires PHP 5.2.5 or higher.
(note, it still may be an issue with PHP 5.2.5, but I can't verify that at this point).
Comment #8
stewart.adam CreditAttribution: stewart.adam commented5.2.17 is greater than 5.2.5...
Comment #9
moonray CreditAttribution: moonray commentedYou are absolutely right. For some reason I read that as 5.2.1.7.
I just found that I can switch from 5.3.x to 5.2.17, so I'm going to be testing this.
Comment #10
moonray CreditAttribution: moonray commentedI can't reproduce the errors on 5.2.17 either.
Can you provide some more info regarding your site's setup? Perhaps a screenshot of the skins admin page might help narrow things down?
Comment #11
stewart.adam CreditAttribution: stewart.adam commentedCome to think about it, I forget which of our sites was showing this glitch... This was a little while ago. I'll try to find more info than just the PHP and OS versions and let you know. In the mean time, I'll leave this as postponed.
Comment #12
moonray CreditAttribution: moonray commentedFinally was able to reproduce this and fix it in multiple places (including some tests).