[MGNLREST-78] PUT request fails to create properties Created: 29/Sep/16 Updated: 10/Nov/16 Resolved: 04/Oct/16 |
|
| Status: | Closed |
| Project: | Magnolia REST Framework |
| Component/s: | services |
| Affects Version/s: | 1.1.2 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Richard Gange | Assignee: | Unassigned |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| 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 |
|
PUT request fails to create properties Sample data:
{
"name": "test",
"type": "mgnl:page",
"path": "/test",
"properties": [
{
"name": "title",
"type": "String",
"multiple": false
},
"nodes": [
{
"name": "test",
"type": "mgnl:page",
"path": "/test/test",
"properties": []
}
]
}
|
| Comments |
| Comment by Mercedes Iruela [ 04/Oct/16 ] |
|
This request is falling because of "nodes", if the request is done with no subnodes and a proper sintax, it works. Here you can see the example tested in 5.4.8 and 5.4.9 (rest framework 1.1.1 and 1.1.2):
{
"name": "mytest",
"type": "mgnl:page",
"path": "/test/mytest",
"properties": [
{
"name": "title",
"type": "String",
"multiple": false,
"values": [
"This is my title"
]
},
{
"name": "mgnl:template",
"type": "String",
"multiple": false,
"values": [
"mtk:pages/basic"
]
}
]
}
This bug is closed as duplicated, because it is actually reporting issue described in MGNLREST-74. |
| Comment by Richard Gange [ 10/Nov/16 ] |
|
Cool, thanks for clearing this up. |