everything goes well:
- I can add terms
- when I enable the Lexicon filter, the terms get marked on all pages

but when I now go to the Lexicon page, with all the terms, I get the following error:
Fatal error: [] operator not supported for strings in /includes/common.inc on line 2313

when I disable the Lexicon filter, that page works again
(but the markings of the terms on the pages are offcourse gone)

This is on a site, soon to be launched - so all help is welcome

CommentFileSizeAuthor
#9 lexicon.module.patch574 bytesxadag

Comments

marty2081’s picture

marty2081 commented

Hi Tim,

Can you supply some more details about the settings that you use so I can reproduce the error?
Do you have the same problem with the 7.x-1.x-dev version?

marty2081’s picture

marty2081 commented
Status: Active » Postponed (maintainer needs more info)
Barfly’s picture

Barfly commented

Hello, i had the same problem:
Version:7.x-1.x-dev
When i go to the lexicon page:
Fatal error:[] operator not supported for strings in \includes\common.inc on line 2313

It happened when "superscript" or "icon term" indicator is activated.
The error disapear when i select "icon+term" or abbr , acronym , cite dfn.

With "superscript" or "icon term" activated the error disapear when the "show only the single term" is selected instead of "advance the whole lexicon to the term"

Voila ^^
hope it will help you

marty2081’s picture

marty2081 commented
Version: 7.x-1.4 » 7.x-1.x-dev
Status: Postponed (maintainer needs more info) » Fixed

Hi Romain,

Thanks for the info. That helped a lot! I found the problem. If there is a description for a vocabulary, the module will print that on the Lexicon page. To permit the safe use of HTML I ran the check_markup() function that resulted in the mentioned error. Since the description is a plain text field I've changed the check_markup() into the filter_xss() function.

The changes are in 7.x-1.x-dev and will be in 7.x-1.5.

Barfly’s picture

Barfly commented

Glad to be helpfull
Btw congrats for your module it's awesome!

Status: Fixed » Closed (fixed)

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

glennnz’s picture

glennnz commented
Status: Closed (fixed) » Active

I'm using 7.8, and this problem exists for me too, exactly as described in #3

Anonymous’s picture

Anonymous (not verified) commented

I'm using 7x 1.8 and experiencing the same issue as #3.

I disabled the "Show lexicon term descriptions." option and the error went away.

This is a site with internationalization. Upgrading to dev version did not fix the error.

xadag’s picture

xadag commented
StatusFileSize
new lexicon.module.patch574 bytes

Hi,

I found the same problem when i try to use this version of lexicon.
I solve the problem with this patch, the problem is in the l function in the .module file, it expect attributes parameters class to be an array and not a single string (when we look at the source code of l)

This patch solve the problem.

marty2081’s picture

marty2081 commented
Status: Active » Fixed

Applied patch in dev branch. Thanks!

Status: Fixed » Closed (fixed)

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