[MGNLRSSAGG-147] RSS Aggregator misses dependency on DAM Created: 30/Jan/14 Updated: 03/Feb/14 Resolved: 03/Feb/14 |
|
| Status: | Closed |
| Project: | Magnolia RSS Aggregator Module |
| Component/s: | None |
| Affects Version/s: | 2.2 |
| Fix Version/s: | 2.2.2 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Magnolia International | Assignee: | Aleksandr Pchelintcev |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | 4.5_to_5_migration, maintenance | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoR: |
Empty
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
When upgrading form 4.5 to 5.x, images like the hackergotchis located in the DMS are not available. The reason is that the module rss aggregator is migrated before the DMS/DAM during the upgrade. Hence the task ChangeDataDmsReferenceToDamMigrationTask is not able to update correctly references and imgUUID are removed from the RSS workspace. The solution simply consists in declaring this dependency in the module descriptor. |
| Comments |
| Comment by Magnolia International [ 30/Jan/14 ] |
|
The logic of getHackergotchi(String feedLink) has to be reviewed as well. It retrieves the UUID from the img property which is wrong, the property imgUUID must be used instead |
| Comment by Daniel Lipp [ 01/Feb/14 ] |
|
We fixed the dependency but according to Nicolas' comment the logic needs to be fixed as well. If it would turn out the logic is already fine we should at least add a comment so everybody is in the picture. |
| Comment by Jan Haderka [ 01/Feb/14 ] |
|
actually the logic of the getHackergotchi(String feedLink) doesn't seem to have any fault. It's more that image uuids should be changed from old dam format (which used img=data & imgUUID=xyz double property format) to new one which looks like img=jcr:xyz |
| Comment by Jan Haderka [ 01/Feb/14 ] |
|
And according to javadoc of info.magnolia.dam.setup.migration.ChangeDataDmsReferenceToDamMigrationTask the above is indeed the case it doesn't handle. Task is capable of converting only simple link or uuidLink format, but not that of DAMHandler from pervious version of STK. |
| Comment by Jan Haderka [ 01/Feb/14 ] |
|
simple workaround that can be used by those affected by the problem is to run extra sweep for rss://feeds//imgUUID property and rename that property to img instead. Newly created feeds are not affected. |