Hi, nice to meet you.

I installed this formatter few hours ago, and i got the error on preview when adding a new node.
It does crached and any other elements are not shown. So. I reviewed code for debugging.

The crash reason is tid index when using free taggin.
If you using taxonomy term with autocomplete tagging widget, Terms will have tid 'autocreate'.
And this module don't consider that index. I will post patch for fix this problem.

CommentFileSizeAuthor
#1 fix_autocreate_term_error-1539014-1.patch685 bytesbarami

Comments

barami’s picture

barami commented
StatusFileSize
new fix_autocreate_term_error-1539014-1.patch685 bytes

Fixed critical error. It available preview with autocomplete tagging widget.
It display autocreate terms to plain text.(seems like contributed taxonomy module).

damiankloip’s picture

damiankloip commented
Priority: Critical » Major
damiankloip’s picture

damiankloip commented

I have just tested this out, and indeed it throws an error. However, I got an error before I had even used textformatter to display the taxonomy, and even with the textformatter module disabled. So I don't think this patch fixes the whole problem. This patch will still be trying to load terms with a tid of 'autocomplete' and I'm not sure if it should. We might want to refactor the code that generates the taxonomy list further?

damiankloip’s picture

damiankloip commented
Status: Active » Fixed

Ok, It makes sense to include this in anyway, as if there is a fix for core, it won't be right now. I changed your patch a little bit as it didn't work properly for saved taxonomy terms begin displayed. The commit is at http://drupalcode.org/project/textformatter.git/commitdiff/56ffcce if you want to check it out. Otherwise, thanks!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

change some mistake