[MGNLTAGS-60] Overwriting templateSelect fieldType definition removes template selection in Pages app Created: 30/Mar/15 Updated: 13/Apr/15 Resolved: 30/Mar/15 |
|
| Status: | Closed |
| Project: | Marketing Tags Manager |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.1.2, 1.2 |
| Type: | Bug | Priority: | Blocker |
| Reporter: | Federico Grilli | Assignee: | Federico Grilli |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||
| Issue Links: |
|
||||||||||||
| Template: |
|
||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||||||
| Date of First Response: | |||||||||||||
| Description |
|
This module's templateSelect overwrites the definition of /modules/pages/fieldTypes/templateSelect in the field type registry thus making it impossible to set a page template (see also attached screenshot). Definition for this module field type should be renamed. We realised about this issue only now because it looks like marketing-tags 1.1 never made it in 5.3.7. Same goes for 1.1.1 version which was released last early February |
| Comments |
| Comment by Federico Grilli [ 30/Mar/15 ] |
|
For 1.1.x branch see commits at https://jira.magnolia-cms.com/secure/bbb.gp.gitviewer.Commit.jspa?commitId=174ac7a91d673d939d8ee9f97ca95743046c00e5&repoId=298&path= and https://jira.magnolia-cms.com/secure/bbb.gp.gitviewer.Commit.jspa?commitId=d577f9f218f03e7ba1356ac04bdf2de34ee78236&repoId=298&path= |
| Comment by Philip Mundt [ 13/Apr/15 ] |
|
The node in the bootstrap file should have been renamed too. |