[MAGNOLIA-2401] Content2BeanUtil.setNodeDatas(Content node, Object bean, String[] excludes) does nothing Created: 30/Sep/08 Updated: 23/Jan/13 Resolved: 07/Oct/08 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | 3.6.4 |
| Fix Version/s: | 4.0 |
| Type: | Bug | Priority: | Major |
| Reporter: | Will Scheidegger | Assignee: | Magnolia International |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | content2bean | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Template: |
|
| Patch included: |
Yes
|
| 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 |
|
When you call Content2BeanUtil.setNodeDatas(Content node, Object bean, String[] excludes) you will note that no node datas are being set. This is due to the fact, that the method does convert the bean into a Map and filters out the values for the keys in excludes but then returns without doing anything else. It should however call Content2BeanUtil.setNodeDatas(Content node, Map map) afterwards. Patch file for the bug fix is attached. |
| Comments |
| Comment by Philipp Bracher [ 07/Oct/08 ] |
|
Patch can be applied, while the bean2content part is still missing in general |
| Comment by Magnolia International [ 07/Oct/08 ] |
|
patch applied on trunk, thanks Will ! |