The code in delicious_nodeapi() calls _delicious_tag_text() for both $node->body and $node->teaser. However, almost always only one of them is the right one to process. The other will be empty.

When this happens, _delicious_tag_text() unnecessarily loops on all the tags passing an empty $text variable to a preg_match().

Attached a patch which does an emptiness check in delicious_nodeapi() before processing.

CommentFileSizeAuthor
delicious_body_teaser_check.patch797 bytesAmrMostafa

Comments

Anonymous’s picture

Assigned: Unassigned »
Status: Needs review » Fixed

Thankyou!

I'll get that committed to the development branch tomorrow.

Status: Fixed » Closed (fixed)

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