Revise implementation of delta tasks
(MAGNOLIA-5571)
|
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | 5.0 |
| Fix Version/s: | 5.2.2 |
| Type: | Sub-task | Priority: | Neutral |
| Reporter: | Roman Kovařík | Assignee: | Roman Kovařík |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | usability | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Template: |
|
||||||||||||||||
| Date of First Response: | |||||||||||||||||
| Description |
|
| Comments |
| Comment by Roman Kovařík [ 13/Jan/14 ] |
|
Reverted until we'll find cause of Hudson bundle failure... |
| Comment by Daniel Lipp [ 24/Jan/14 ] |
|
Ok, many of those changes are very basic but I'd at least expect a few unit-tests for the new constructors: e.g. for PropertyExistsDelegateTask and IsModuleInstalledOrRegistered After all we're adding a lot of code with this commit and we should make sure there's no bugs in. On top I'd expect a linked ticket to actually make use of these constructors: I'm pretty sure there's many places in our codebase where it would be handy. After all that was the motivation to add the new stuff, no |
| Comment by Roman Kovařík [ 27/Jan/14 ] |
|
| Comment by Jaroslav Simak [ 28/Jan/14 ] |
|
| Comment by Roman Kovařík [ 29/Jan/14 ] |
|
BootstrapSingleResource should not be deprecated. |