Function "theme_glossary_overview_item" uses the "taxonomy_term_count_nodes" function, which is known to be a potential performance bottleneck. The point of this call is merely to know if there is a detailed description (full) node supplied for the term. I believe it would be much quicker to simply run a COUNT query on the term_node table to see if any nodes are tagged with the term.

The supplied patch does this.

CommentFileSizeAuthor
#4 glossary12.txt697 bytesnancydru
glossary12.txt650 bytesnancydru

Comments

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Active » Needs review
moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Needs work

you forgot status=1 in the node table. right?

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

I was trying to keep it very simple and fast. Of course I can join the node table to check that. I wonder if it makes a difference to anyone.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Assigned: Unassigned » nancydru
StatusFileSize
new glossary12.txt697 bytes
nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Needs work » Needs review
nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Needs review » Fixed

Committed

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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