Here is a patch that intends to fix three bugs:

  • The major one: useless terms do not show anymore in the selection list: when the user selects a term, there is always at least one selectable node.
  • Corrected a wrong usage of the t() function.
  • Added a strip_tags() to the output of _nodereference_item() when filling the list of selectable nodes, to cut off the noisy HTML that this function may produce.

Anyone to test and review this?

CommentFileSizeAuthor
#4 taxonomySearch.module.patch2.48 KBricflomag
#1 taxonomySearch.module.patch2.17 KBricflomag
taxonomySearch.module.patch2.02 KBricflomag

Comments

ricflomag’s picture

ricflomag commented
StatusFileSize
new taxonomySearch.module.patch2.17 KB

New version of the patch that fixes a problem with the indexes of the of the options in the taxonomy select box, and allows no filtering at all when the "Please choose" option is selected (instead of having an empty node selection list).

Still anyone to review it ?

markfoodyburton’s picture

markfoodyburton commented

THANKS for the patch, I'm going to commit it, and release a new version of the module, with the content_taxonomy listed as a dependency

ricflomag’s picture

ricflomag commented

Glad to read this. Thanks.

ricflomag’s picture

ricflomag commented
StatusFileSize
new taxonomySearch.module.patch2.48 KB

Slightly modified patch:

* Removed the strip_tags() function on _nodereference_item() return value, replaced by setting the third argument of this function to FALSE, so that it does not return HTML, but plain text.
* Done on both calls to _nodereference_item() (one had been omitted).

Any chance to release a new version soon ? ;)