Old flag table references in Message subscribe UI [#2551485]

In message_subscribe_ui/message_subscribe_ui.views_default.inc there are various references to an old flag table named 'flag_content'.
That table has been renamed to 'flagging' in flag 7.x-3.6 and this makes message_subscribe_ui feature overridden.

Comment	File	Size	Author
#13	old_flag_table-2551485-13.patch	8.68 KB	jlscott
#13	7.x-1.x: PHP 5.3 & MySQL 5.5, D7 16 pass
#10	old_flag_table-2551485-10.patch	8.89 KB	jlscott
#10	7.x-1.x: PHP 5.3 & MySQL 5.5, D7 Patch Failed to Apply
#8	old_flag_table-2551485-8.patch	9.27 KB	jlscott
#8	7.x-1.x: PHP 5.3 & MySQL 5.5, D7 Patch Failed to Apply
#5	old_flag_table-2551485-5.patch	4.78 KB	kmajzlik
#5	7.x-1.x: PHP 5.3 & MySQL 5.5, D7 16 pass
#2	message_subscribe-fix_old_flag_table_references-2551485-0.patch	4.39 KB	imfaber
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

14 August 2015 at 14:03

imfaber created an issue. See original summary.

Comment #2

imfaber CreditAttribution: imfaber at Edo commented 14 August 2015 at 14:09

Status:

Active

» Needs review

File	Size
message_subscribe-fix_old_flag_table_references-2551485-0.patch	4.39 KB

Here is a patch that brings the feature back to its default status.

Comment #3

14 August 2015 at 14:14

Status:

Needs review

» Needs work

The last submitted patch, 2: message_subscribe-fix_old_flag_table_references-2551485-0.patch, failed testing.

Comment #4

imfaber CreditAttribution: imfaber at Edo commented 26 August 2015 at 08:53

Status:

Needs work

» Closed (won't fix)

Comment #5

kmajzlik CreditAttribution: kmajzlik at VMWeb s.r.o. for SomvPrahe commented 16 November 2016 at 17:05

Status:

Closed (won't fix)

» Needs review

File	Size
old_flag_table-2551485-5.patch	4.78 KB
7.x-1.x: PHP 5.3 & MySQL 5.5, D7 16 pass

This has to be fixed and not closed due to wrong patch.
I just recreated patch to be compatible with testing bot.

Overriden Features are boring when we are not able to revert them.

Comment #6

igorik CreditAttribution: igorik as a volunteer commented 16 November 2016 at 18:10

Status:

Needs review

» Reviewed & tested by the community

It works perfectly.
Thank you!

Comment #7

kmajzlik CreditAttribution: kmajzlik at VMWeb s.r.o. for SomvPrahe commented 16 November 2016 at 18:56

Assigned:

imfaber

» Unassigned

Comment #8

jlscott CreditAttribution: jlscott as a volunteer commented 19 February 2017 at 09:15

File	Size
old_flag_table-2551485-8.patch	9.27 KB
7.x-1.x: PHP 5.3 & MySQL 5.5, D7 Patch Failed to Apply

The patch in #5 needs to be updated to also include the views in file "message_subscribe_email.views_default.inc" of the message_subscribe_email sub-module. Updated patch attached.