[MGNLPN-333] CookieFieldTransformer doesn't store values when multiple cookies are available Created: 21/Sep/16 Updated: 31/Jan/17 Resolved: 27/Jan/17 |
|
| Status: | Closed |
| Project: | Magnolia Personalization |
| Component/s: | Traits |
| Affects Version/s: | 1.3 |
| Fix Version/s: | 1.2.9, 1.3.2, 1.4.2 |
| Type: | Bug | Priority: | Major |
| Reporter: | Antti Hietala | Assignee: | Evzen Fochr |
| 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
|
||||||||
| Sprint: | Kromeriz 81 | ||||||||
| Story Points: | 3 | ||||||||
| Description |
|
CookieFieldTransformer extends SwitchableTransformer but it doesn't operate like a real switchable transformer. When two or more cookies are available in the cookies trait, such as in the travel demo, the CookieFieldTransformer only stores the first cookie's value and ignores the second cookie's value. To reproduce in the travel demo:
Result:
If you change the transformer class of the cookies trait ruleField to a generic SwitchableTransformer, Magnolia stores both cookie values correctly but doesn't use the special property names cookieName and cookieValue. I set the priority to major since this issue prevents you from using more than one cookie in the same variant. Findings: |