[MGNLCDEP-28] Dependency Chart throws FreeMarker error on nonexisting page Created: 13/Dec/12 Updated: 25/Jan/13 Resolved: 15/Jan/13 |
|
| Status: | Closed |
| Project: | Content Dependencies |
| Component/s: | None |
| Affects Version/s: | 1.0.5 |
| Fix Version/s: | 1.1 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Roman Kovařík | Assignee: | Roman Kovařík |
| 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 |
|
dependencies.html -> enter any nonexisting page -> Generate chart: get(moo) failed on instance of info.magnolia.module.dependencies.pages.DependenciesPage
The problematic instruction:
----------
==> ${this.moo} [on line 42, column 32 in info/magnolia/module/dependencies/pages/DependenciesPage.html]
----------
|
| Comments |
| Comment by Jan Haderka [ 14/Jan/13 ] |
|
I don't think that ignoring the RepositoryException is really good solution. You can do so for ItemNotFoundException or PathNotFoundException ... in which case template should show some message saying that specified content was not found and you should show the error message and print stacktrace in the log file for all other kinds of RepositoryException errors. |