Closed (fixed)
Project:
Hierarchical Select
Version:
6.x-3.x-dev
Component:
Code
Priority:
Minor
Category:
Task
Assigned:
Reporter:
Created:
18 Aug 2009 at 22:29 UTC
Updated:
14 Apr 2010 at 20:53 UTC
Jump to comment: Most recent file
please add
error_reporting(E_ALL);
ini_set('display_errors', TRUE);
ini_set('display_startup_errors', TRUE);
to your settings.php to show all these PHP warnings (if your server is not already configured to do so) and test a bit around.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 552910-2.patch | 17.75 KB | wim leers |
| hierarchical_select-fix-php-notices.patch | 5.67 KB | eMPee584 |
Comments
Comment #1
wim leersConfiguring settings.php is not sufficient apparently (I had done that a long time ago already and had given up because I didn't get to see any notices).
You must also patch Drupal core: it won't show E_NOTICE errors out-of-the-box. Stupid!
I'll review your patch ASAP :)
Comment #2
wim leersI fixed all notices that I could find/produce. I'm sure there are more, although most notices should be fixed now. I've based my patch on your changes, but changed some of your modifications and added many, many more fixes.
http://drupal.org/cvs?commit=281808
Comment #3
wim leersOh and the changes to core that are necessary: in common.inc, change:
if ($errno & (E_ALL ^ E_NOTICE ^ E_DEPRECATED)) {to:
if ($errno & (E_ALL ^ E_NOTICE | error_reporting())) {Comment #4
wim leersDammit, this patch consists of very simple changes and therefor I went ahead and committed it. But apparently this broke HS.
Comment #5
wim leersFixed again: http://drupal.org/cvs?commit=281956. Also fixed another notice at the same time.
Comment #6
jrao commentedThis fix introduced an error in hs_taxonomy.module:
I think you meant to use:
Otherwise the parent and related terms in new terms form will always be empty.
Comment #7
wim leersGood find. Thanks! Will commit ASAP.
Comment #8
wim leersjrao, sorry for not committing your fix right away. This was solved in #640418: Unable to select parent terms on add term form though, many weeks ago. I hope you saw this was "suddenly" fixed :)