Download & Extend
Content Taxonomy » Issues

Fix PHP notices

Posted by joelstein on March 2, 2011 at 7:03pm
Project:Content Taxonomy
Version:6.x-1.0-rc2
Component:Documentation
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs work
Issue tags:php5.3

Issue Summary

The attached patch fixes it.

AttachmentSize
content_taxonomy.module.patch615 bytes
Login or register to post comments

Comments

#1

Posted by joelstein on March 2, 2011 at 7:18pm
Title:Notice: Undefined index: op in content_taxonomy_form_alter» Fix PHP notices
Component:Code» Documentation

I found a couple more.

AttachmentSize
content_taxonomy.module.patch 1.93 KB

#2

Posted by vectoroc on September 22, 2011 at 1:52pm

thanks

#3

Posted by fgm on October 2, 2011 at 2:48pm
Status:needs review» reviewed & tested by the community

I was just going to suggest the exact same patch: good for me.

Same problem still exists in 6.x-2.x, by the way.

#4

Posted by fgm on October 2, 2011 at 2:49pm

Tagging for 5.3.

#5

Posted by fgm on October 2, 2011 at 2:57pm
Version:6.x-1.x-dev» 6.x-1.0-rc2
Status:reviewed & tested by the community» needs review

Except of course it needs to be rerolled in git format against an actual version, so someone else needs to check.

AttachmentSize
missing_isset_notices-1079048-5.patch 1.81 KB

#6

Posted by vectoroc on October 10, 2011 at 1:21pm

here is a patch for another notices

AttachmentSize
content_taxonomy_notices.patch 1.44 KB

#7

Posted by vectoroc on October 10, 2011 at 1:51pm

and another set of fixes

AttachmentSize
content_taxonomy_notices2.patch 1.59 KB

#8

Posted by fgm on October 10, 2011 at 5:26pm
Status:needs review» needs work

if (isset($entry['value']) && $entry['value']) can be replaced by if (!empty($entry['value']) which is slightly shorter.

nobody click here