Project

General

Profile

Actions

Bug #8183

closed

Edit of Post creates notification emails

Added by Matt Gold almost 7 years ago. Updated over 6 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
Group Blogs
Target version:
Start date:
2017-05-22
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

Hi Boone,

I think that Bug 6350 is cropping up again. An editor of JITP edited an old post and a notification was sound out -- one showing it as getting published, the other as getting edited. Please see the notification email attached


Files

original_msg (11).txt (38.7 KB) original_msg (11).txt Matt Gold, 2017-05-22 01:24 PM

Related issues

Related to CUNY Academic Commons - Bug #6350: Editing blog posts results in activity itemsResolvedBoone Gorges2013-10-22

Actions
Related to CUNY Academic Commons - Bug #2844: Commons bug re: editing blog postsResolvedBoone Gorges2013-10-22

Actions
Related to CUNY Academic Commons - Bug #8535: Blog post edit triggers group notificationResolvedRaymond Hoh2017-08-15

Actions
Related to CUNY Academic Commons - Bug #14017: Editing of blog post leads to group notificationResolvedRaymond Hoh2021-02-21

Actions
Actions #1

Updated by Matt Gold almost 7 years ago

  • Related to Bug #6350: Editing blog posts results in activity items added
Actions #2

Updated by Matt Gold almost 7 years ago

  • Related to Bug #2844: Commons bug re: editing blog posts added
Actions #3

Updated by Boone Gorges almost 7 years ago

This is not exactly a recurrence of #6350, though the filter introduced in #6350 accidentally caused the problem in this ticket to be covered up. The workaround in #6350 was rolled back as part of #7683.

Briefly, this problem only exists for legacy posts. A number of years ago (it appears to be May 2013 or so), bp-groupblog was modified so that groupblog activity items had the type 'new_groupblog_post' rather than BP's default 'new_blog_post'. Posts created before this date (actually, most posts - some have new_groupblog_post, for reasons I don't recall) have 'new_blog_post' activity items. When bp-groupblog looks for existing groupblog activity items corresponding to the currently edited post, it looks for 'new_groupblog_post' items. As such, it misses the existence of the existing activity item, and creates a new one.

Ray - Rather than writing a kludge workaround, my inclination is do something like this:

UPDATE wp_bp_activity SET type = 'new_groupblog_post' WHERE type = 'new_blog_post' AND component = 'groups'

Can you spent 60 seconds thinking about whether this would have any weird consequences? I'm 95% sure it's safe.

Actions #4

Updated by Raymond Hoh almost 7 years ago

Sounds safe to me.

Maybe the cac_block_groupblog_activity_bump() function can be removed after the DB query is run?

Actions #5

Updated by Boone Gorges almost 7 years ago

  • Category name set to Group Blogs
  • Status changed from Assigned to Resolved
  • Target version set to 1.11.2

Thanks, Ray. I've run the query, which should fix this ticket.

I think you're right that that function is no longer needed, though to be honest, I'm unsure why it didn't prevent what Matt describes in this ticket. Not worth further investigation, I guess. https://github.com/cuny-academic-commons/cac/commit/3cccfa927d17f394500b33943ea6aebde0538c01

Actions #6

Updated by Boone Gorges almost 7 years ago

  • Related to Bug #7997: Transition from single cac-files.php file server to site-specific .htaccess directives added
Actions #7

Updated by Boone Gorges almost 7 years ago

  • Related to deleted (Bug #7997: Transition from single cac-files.php file server to site-specific .htaccess directives)
Actions #8

Updated by Matt Gold almost 7 years ago

Thanks to you both!

Actions #10

Updated by Boone Gorges over 6 years ago

  • Related to Bug #8535: Blog post edit triggers group notification added
Actions #11

Updated by Boone Gorges over 6 years ago

Luke - I've opened #8535 to track the new issue, which seems similar to this one on the surface but appears to have a different cause.

Actions #12

Updated by Matt Gold about 3 years ago

  • Related to Bug #14017: Editing of blog post leads to group notification added
Actions

Also available in: Atom PDF