Steps to repeat:
1. Install fbss and fbss_notifications
2. Create a status, delete it

Results:
user warning: Unknown table 'notifications' in MULTI DELETE query: DELETE notifications, notifications_fields FROM notifications n INNER JOIN notifications_fields f ON n.sid = f.sid WHERE (n.event_type = 'status' AND f.intval = 3336) OR (n.event_type = 'stream' AND f.intval = 371) in /mnt/www/html/i/docroot/profiles/drupal_commons/modules/contrib/facebook_status/submodules/fbss_notifications/fbss_notifications.module on line 77.

Expected results:
The values are deleted.

Patch coming...

CommentFileSizeAuthor
#1 1557840_use_aliases_everywhere.patch952 bytesgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs review
Issue tags: +commonslove
StatusFileSize
new 1557840_use_aliases_everywhere.patch952 bytes

Patch attached. I changed to use the alias everywhere and to use an alias for notifications that won't be confused for the alias typically used for the node table (e.g. by db_rewrite_sql).

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

I hadn't tested the patch in #1, but have done so now and it fixes the bug for me.

icecreamyou’s picture

icecreamyou commented
Status: Needs review » Fixed

Committed the patch from #1 to dev (and D7 as well). Thanks!

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