[MGNLUI-5560] Exporting node with short node name fails Created: 12/Dec/19  Updated: 13/Feb/20  Resolved: 12/Feb/20

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 6.2
Fix Version/s: 6.2

Type: Bug Priority: Neutral
Reporter: Christoph Meier Assignee: Rishab Dhar
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:
Sprint: UI Framework 16
Story Points: 3

 Description   

In the new UI framework, Exporting nodes with short node name fails, if the length of the node name is shorter than 3 characters.

To reproduce

  • In the pages app, create the subtree /a/b/c, publish it
  • Try to export: Fails on every level
  • rename "a" -> "aa" => export still fails
  • rename "aa" -> "aaa" => export works

stack-trace / gist

 Please add unit test for this case!

 Please note that the export works when using the JCR app with short names.
JCR app is using the old UI framework.

Couple of things to investigate:

  • one of the symptoms/causes of this bug is that in assets detail app the asset file name is displayed and is resolved from the corresponding temp file name. "Enhancing" the temp file name with random suffix/prefix will spoil the looks in the form. Can we untie the displayed file name from the temp file?
  • do we need to display that file name in the asset form at all? (does it leave any footprint in the resulting data, is it important/useful at all, don't typical file upload services not care about the source file name?)

 



 Comments   
Comment by Rishab Dhar [ 12/Feb/20 ]
  • Use TempFilesManager API instead for creating tempFile for export
  • Unit tests are part of TempFilesManager in MGNLUI-5638
Generated at Mon Feb 12 09:27:47 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.