[MSHOP-22] The demo-project is requested when adding the shop module to my application Created: 17/Jan/12 Updated: 19/Mar/13 Resolved: 03/May/12 |
|
| Status: | Closed |
| Project: | Magnolia Shop (closed) |
| Component/s: | None |
| Affects Version/s: | 1.0.1 |
| Fix Version/s: | 1.0.7 |
| Type: | Bug | Priority: | Blocker |
| Reporter: | Fabrice LAZZARI | Assignee: | Teresa Miyar |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| 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)
|
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
| Date of First Response: |
| Description |
|
I've downloaded the shop module from the magnolia shop, and installed it in the WEB-INF\lib of my application. When starting the application I get the following error message, saying that I need the demo-project. It's OK for test purpose, but I'd like to be able to install the shop on our productive system without demo-project => blocker issue. Could you remove the dependency ? Thanks ! -------------------------------------- java.lang.Exception: Exception received from deployment driver. See Error Log view for more detail. |
| Comments |
| Comment by Will Scheidegger [ 17/Jan/12 ] |
|
You can actually run the shop without demo-project installed. It's just the installer which requires demo-project to be around at install time. @Theresa: Can the install task check if demo-project is installed and only install the samples if it is? |
| Comment by Teresa Miyar [ 17/Jan/12 ] |
|
Will, I just read your comment, i did set it to optional in the module descriptor, and if you set bootstrapsamples to false then the samples bootstrap will be ignored. Fell free to reopen the issue if you think otherwise |
| Comment by Teresa Miyar [ 17/Jan/12 ] |
|
Will, i think you are right, will check if the module is installed as well |
| Comment by Will Scheidegger [ 17/Jan/12 ] |
|
Wow, that was quick! Thanks! |
| Comment by Fabrice LAZZARI [ 23/Jan/12 ] |
|
Thanks ! Looking now forward to getting a new release build. |
| Comment by Ondrej Chytil [ 26/Apr/12 ] |
|
Needs to be backported to branch. |