Closed (outdated)
Project:
Drupal core
Version:
7.x-dev
Component:
forum.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
17 Jan 2012 at 19:02 UTC
Updated:
9 Aug 2022 at 20:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Zgear commenteddrupal 8.x first, I reproduced a similar result by changing the taxonomy field to use the autocomplete widget instead of a list, then I created a forum and placed a comma into the field. The problem seems to lie with the variable $tid, although mine came up with $forum_tid. Perhaps we have a problem in how we are handling taxonomies in the forum module?
Comment #2
Zgear commentedcredit goes to zendoodles for thinking of this way to reproduce
STR
you should come up with something like this
Comment #3
berdirComment #4
cweagansUpdating tags per http://drupal.org/node/1517250
Comment #5
tillsanders commentedTried that one? Worked for me:
http://drupal.org/node/1057712#comment-4733236
Comment #6
cweagansPlease don't change the issue metadata.
Comment #14
thursday_bw commentedGive this change record https://www.drupal.org/node/2079767 where the function in question on this error doesn't exist in drupal 8 anymore.
Can we re-action the change of the version on this issue, marking it as drupal 7.
I understand there is a reference to this being replicated in drupal 8 but that was prior to this change record.
Comment #16
quietone commentedI tested this on Drupal 9.3.x, standard install, using the steps in #2, and was not able to reproduce the problem. On saving the forum topic I received, "the value should not be null".
Since this is tagged for backport and I have not tested with Drupal, changing version.
Comment #17
poker10 commentedIt seems like there are two different problems mentioned in this issue.
The first one is when you try to pass empty string TID to the
forum_forum_load()function. This will cause notices and warnings mentioned in issue summary. See: https://www.drupal.org/node/1057712The second problem is mentioned in #2 - these steps are not steps to reproduce the original problem, but the separate issue, when you try to save forum topic with forum reference value set to comma. This will not trigger the notices mentioned in issue summary, but different notices.
I think this issue should focus on fixing the problem mentioned in the issue summary and potentially the next issue should be created to fix the problem with saving forum topic with forum reference set to comma - which will (on PHP 8.1) throw a TypeError: