Make RSS Aggregator module compatible with Magnolia 5.
(MGNLRSSAGG-101)
|
|
| Status: | Closed |
| Project: | Magnolia RSS Aggregator Module |
| Component/s: | None |
| Affects Version/s: | 2.2 |
| Fix Version/s: | 2.2 |
| Type: | Sub-task | Priority: | Neutral |
| Reporter: | Aleksandr Pchelintcev | Assignee: | Roman Kovařík |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Template: |
|
||||||||||||||||
| Date of First Response: | |||||||||||||||||
| Sprint: | 5.2-rc2 | ||||||||||||||||
| Comments |
| Comment by Federico Grilli [ 28/Oct/13 ] |
|
I think update tasks can be simplified, i.e. .addTask(new NodeExistsDelegateTask("Set property", "Set workspace for rss importer", RepositoryConstants.CONFIG, "/modules/data/config/importers/rssaggregator", new PropertyExistsDelegateTask("", "", RepositoryConstants.CONFIG, "/modules/data/config/importers/rssaggregator", "repository", new SetPropertyTask("", RepositoryConstants.CONFIG, "/modules/data/config/importers/rssaggregator", "repository", RSSAggregatorConstants.WORKSPACE), null) one could omit the check for parent node existence which already done by PropertyExistsDelegateTask .addTask(new PropertyExistsDelegateTask("Do something", "Does something.", RepositoryConstants.CONFIG, "/modules/data/config/importers/rssaggregator", "repository", new SetPropertyTask("", RepositoryConstants.CONFIG, "/modules/data/config/importers/rssaggregator", "repository", RSSAggregatorConstants.WORKSPACE)) |
| Comment by Roman Kovařík [ 08/Nov/13 ] |
|