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 |
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 commentedAttached patch resolves issue.
Comment #2
rogical commentedIt works!
Comment #3
stewart.adam commentedSetting RTBC :)
Comment #4
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 commentedI can't reproduce this issue @stewart.adam. What version of PHP are you using?
Comment #6
stewart.adam commentedPHP 5.2.17 (CentOS 5)
Comment #7
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 commented5.2.17 is greater than 5.2.5...
Comment #9
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 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 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 commentedFinally was able to reproduce this and fix it in multiple places (including some tests).