[MGNLNEWSLETTER-4] Fix renaming issue Created: 04/Jun/09  Updated: 16/Sep/15  Resolved: 04/Jun/09

Status: Closed
Project: Newsletter (closed)
Component/s: api, stk integration
Affects Version/s: 1.0 RC1
Fix Version/s: 1.0 RC2

Type: Bug Priority: Major
Reporter: Magnolia International Assignee: Magnolia International
Resolution: Fixed Votes: 0
Labels: None
Σ Remaining Estimate: Not Specified Remaining Estimate: Not Specified
Σ Time Spent: Not Specified Time Spent: Not Specified
Σ Original Estimate: Not Specified Original Estimate: Not Specified

Issue Links:
dependency
depends upon MAGNOLIA-2762 ModuleRegistry : throw an exception f... Closed
supersession
is superseded by MAGNOLIA-2761 ModuleRegistry: retrieve module by class Closed
Sub-Tasks:
Key
Summary
Type
Status
Assignee
MGNLNEWSLETTER-20 Retrieve module by class Sub-task Closed Magnolia International  
Template:
Acceptance criteria:
Empty
Date of First Response:

 Description   

Just before RC1, i renamed the newsletter module to newsletter-api (in the module descriptor). I stupidly forgot to update the code that used this.
Not convinced newsletter-api is the best name either, but at least it's consistent with the maven artifacts, and (sort of) conveys the fact that the module on its own is useless. (it needs an impl.)



 Comments   
Comment by Magnolia International [ 04/Jun/09 ]

So the module is now called newsletter. One of the reasons is because this is where you configure the actual implementation.
Unfortunately this is inconsistent with the maven module name... but I'm not too keen on changing that name.

The impl. still needs to be a module (it could have been just a jar) because it needs a version handler to register itself.

Comment by Michael Mühlebach [ 16/Sep/15 ]

This ticket was closed because former resolved tickets are deemed to be closed now. If this assumption is untrue in this particular case please feel free to reopen the ticket again.

Generated at Mon Feb 12 10:21:54 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.