Synonyms 7.x-1.5 integration [#2728743]

Current Term Merge 7.x dev snapshot fails with bundled with latest Synonyms release (that being Synonyms 7.x-1.5).

Comment	File	Size	Author
#2	2728743-synonyms-integration-2.patch	18.52 KB	bucefal91
#2	7.x-1.x: PHP 5.3 & MySQL 5.5, D7 23 pass PHP 5.4 & MySQL 5.5, D7 23 pass PHP 5.5 & MySQL 5.5, D7 23 pass PHP 5.6 & MySQL 5.5, D7 23 pass

Comments

18 May 2016 at 18:31

bucefal91 CreditAttribution: bucefal91 commented 18 May 2016 at 20:06

Status:

Active

» Needs review

File	Size
2728743-synonyms-integration-2.patch	18.52 KB
7.x-1.x: PHP 5.3 & MySQL 5.5, D7 23 pass PHP 5.4 & MySQL 5.5, D7 23 pass PHP 5.5 & MySQL 5.5, D7 23 pass PHP 5.6 & MySQL 5.5, D7 23 pass

Here is the patch for this issue. It does the following:

Defines a new synonyms behavior "term_merge" and implements it for all known field-based synonyms providers. We need a stand alone behavior for this purpose now, because there are entity properties, that do not support merging terms into them, as they are read-only for our purpose of synonyms.
Updates simpletests to keep up with the code.

I'll let it be here for a few days and then will commit it. Locally, I have 100% pass on simpletests.

26 May 2016 at 01:08

bucefal91 committed 04c9758 on 7.x-1.x

Issue #2728743 by bucefal91: Making module compatible with Synonyms 7.x-...

bucefal91 CreditAttribution: bucefal91 commented 26 May 2016 at 01:08

Status:

Needs review

» Fixed

Committed!

9 June 2016 at 01:14

Status:

Fixed

» Closed (fixed)

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