[MGNLOBS-29] RestrictToNodeTypeEventListener looks for rep:root, not the configured node type, then returns null path warning Created: 07/May/14 Updated: 27/May/14 Resolved: 16/May/14 |
|
| Status: | Closed |
| Project: | Observation |
| Component/s: | None |
| Affects Version/s: | 2.0.1 |
| Fix Version/s: | 2.0.2 |
| Type: | Bug | Priority: | Major |
| Reporter: | Antti Hietala | Assignee: | Federico Grilli |
| 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
|
| Testcase included: |
Yes
|
| Description |
|
RestrictToNodeTypeEventListener throws a "null path" warning when used with the example listener config sendMailOnPageChanges. WARN servation.commands.RestrictToNodeTypeEventListener: Returning null path as either node was null or actual node type [rep:root] is not equal to the configured type [mgnl:content]. The listener config only observes changes to page properties such as page name. It is fair to assume that users would expect it to also observe changes to areas and components. Editing an area or component should be considered a "page change" as the listener name implies. Please fix either the listener class or the example config so that they work out of the box. To reproduce:
Possible solutions:
Whichever route you take, please add includeSubNodes=true to the sendMailOnPageChanges listener config so that edits to child pages are also detected. |