There are a number of PHP warnings on content type fields pages and node edit pages due to undefined variables and the like.

CommentFileSizeAuthor
#2 conditional_fields-1096628-2-fix_php_warnings.patch10.93 KBarithmetric
#1 conditional_fields-1096628-fix_php_warnings.patch9.3 KBarithmetric

Comments

arithmetric’s picture

arithmetric commented
Status: Active » Needs review
StatusFileSize
new conditional_fields-1096628-fix_php_warnings.patch9.3 KB

Here's a patch to fix the instances that I've encountered.

arithmetric’s picture

arithmetric commented
StatusFileSize
new conditional_fields-1096628-2-fix_php_warnings.patch10.93 KB

I've updated the patch with one more fix for a PHP warning, and a fix for one of the fixes in the last patch.

jaxxed’s picture

jaxxed commented
Priority: Normal » Major

upping priority. These issues are significant, as in some cases I've had them add over 100 entries to watchdog per page load.

Arithmetic: Have you field tested your patches?

Module-developer: isset(), empty()

YK85’s picture

YK85 commented

subscribing

jaxxed’s picture

jaxxed commented

Subscribers should test the supplied patch. If the code is reviewed then we can try to submit it.

peterpoe’s picture

peterpoe commented
Status: Needs review » Fixed

Patch committed. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.