bookmarks.admin.inc line 729:
updated FROM menu_links WHERE mlid=%d
should be:
updated FROM {menu_links} WHERE mlid=%d

bookmarks.admin.inc line 763:
SELECT mlid FROM menu_links WHERE plid=%d
should be:
SELECT mlid FROM {menu_links} WHERE plid=%d

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Postponed

D'oh.

I should really stop committing code in the middle of the night.

I'll make sure this gets fixed in the next release. Thanks for catching it.

AlexisWilke’s picture

AlexisWilke commented
Status: Fixed » Postponed

In the version I have there is a missing { } set for mlid and it happens to be on line 763 too.

$result = db_query("SELECT mlid FROM menu_links WHERE plid=%d", $item['mlid']);

when the menu_links is actually correct and is on line 758...

Thank you.
Alexis Wilke

Anonymous’s picture

Anonymous (not verified) commented
Status: Postponed » Fixed

Fixed in 6.x-2.4.

Status: Postponed » Closed (fixed)

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