[magnolia-dev] [JIRA] (MGNLUI-3863) After creating nodes with (invalid) characters, such as commercial/at/@ Magnolia apps might crash when addressing such nodes

[JIRA (on behalf of Jan Haderka)]

Title: Message Title

Jan Haderka updated an issue   Magnolia UI / MGNLUI-3863 After creating nodes with (invalid) characters, such as commercial/at/@ Magnolia apps might crash when addressing such nodes Change By: Jan Haderka Summary: Most apps allow you to create After creating  nodes with  (  invalid )  characters, such as commercial /  at  ( / @ )  Magnolia apps might  crash when addressing such nodes Account: null (null) Commercial at (@) is currently forbidden character in node name. Forbidden by Magnolia that is, for historical reasons, see more at https://documentation.magnolia-cms.com/display/DOCS/Managing+pages#Managingpages-Invalidcharacters. Apps should not allow Due  to  create nodes with @  those characters being historically forbidden by Magnolia, there are various parts that do not handle such chars well as it was not required  in  name except  the  past. Said commercial character is however no longer treated consistently and most apps allow it's use as node or property name. Without changing functionality of Magnolia where commercial is allowed on purpose (  {{user}} subapp ), we need to treat nodes with commercial consistently .  Therefore: *  {{  AddNodeAction  action }}  should validate label name and not create node with invalid name *  {{  SaveRoleDialogAction }}  should create roles with valid node names *  {{  JcrContentConnector }}  should not fail on NPE when node contains @ *  remove duplicated code between {{  JcrContentConnector  duplicates method from }} and {{  JcrItemUtil }} Once we MGNLUI-322 is resolved, commercial character can be enabled again for all apps, by simply removing it from the validation method and enabling it in all places at once. Add Comment   This message was sent by Atlassian JIRA (v6.4.13#64028-sha1:b7939e9) ---------------------------------------------------------------- For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html Alternatively, use our forums: http://forum.magnolia-cms.com/ To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com> ----------------------------------------------------------------