[MAGNOLIA-2882] Introduce vararg constructors in ArrayDelegateTask Created: 07/Oct/09 Updated: 23/Jan/13 Resolved: 07/Oct/09 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core, updatemechanism |
| Affects Version/s: | None |
| Fix Version/s: | 4.2 |
| Type: | Improvement | Priority: | Trivial |
| Reporter: | Magnolia International | Assignee: | Magnolia International |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| 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)
|
||||
| Date of First Response: | |||||
| Comments |
| Comment by Philipp Bärfuss [ 08/Oct/09 ] |
|
The drawback of this is that older version handlers are not binary compatible any more: Take the data module as an example: SEVERE: Exception sending context initialized event to listener instance of class info.magnolia.cms.servlets.MgnlServletContextListener java.lang.NoSuchMethodError: info.magnolia.module.delta.ArrayDelegateTask.<init>(Ljava/lang/String;Ljava/lang/String; \ Linfo/magnolia/module/delta/Task;Linfo/magnolia/module/delta/Task; \ Linfo/magnolia/module/delta/Task;Linfo/magnolia/module/delta/Task;)V at info.magnolia.module.data.setup.DataModuleVersionHandler.<init>(DataModuleVersionHandler.java:108) at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) |
| Comment by Magnolia International [ 08/Oct/09 ] |
|
oops, you're right. re-adding the "fixedargs" constructors. |