Comments

joelstein’s picture

Title: Notice: Undefined index: op in content_taxonomy_form_alter » Fix PHP notices
Component: Code » Documentation
StatusFileSize
new1.93 KB

I found a couple more.

vectoroc’s picture

thanks

fgm’s picture

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.

fgm’s picture

Issue tags: +php5.3

Tagging for 5.3.

fgm’s picture

Version: 6.x-1.x-dev » 6.x-1.0-rc2
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new1.81 KB

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

vectoroc’s picture

StatusFileSize
new1.44 KB

here is a patch for another notices

vectoroc’s picture

StatusFileSize
new1.59 KB

and another set of fixes

fgm’s picture

Status: Needs review » Needs work

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

gateway69’s picture

any updates to this? we are seeing this still

Notice: Undefined index: op in content_taxonomy_form_alter() (line 333 of /var/www/sites/all/modules/content_taxonomy/content_taxonomy.module).
hchonov’s picture

Issue summary: View changes
StatusFileSize
new1.44 KB

Couldn't apply content_taxonomy_notices2.patch because of the path prefixes, which have been exchanged with a/ and b/ in content_taxonomy_notices2.1.patch