EZP-31742: UDW error for users with restricted content types

[SerheyDolgushev]

Question	Answer
Tickets	https://jira.ez.no/browse/EZP-31742
Bug fix?	yes
New feature?	no
BC breaks?	no
Tests pass?	yes
Doc needed?	no
License	GPL-2.0

Steps to reproduce:

1. Install clean eZ Platform 3.1.0/3.0.5.
2. Modify editor user role (https://ezplatform310.localhost/admin/role/3):
   • Remove "Content / All functions" limitation
   • Add "Content / Read, Content Type: Folder" limitation
   • Add "Content / Create, Content Type: Folder" limitation
3. Create a test editor user.
4. Login into admin using test editor user credentials.
5. Open "Content Structure" page (https://ezplatform310.localhost/admin/view/content/1/full/1/2).
6. Click "Browse" button in the left menu (div.ez-sticky-container).

Expected result:
UDW is opened

Actual result:
JavaScript error is throw in the browser console

So UDF has serious issues for the users with "Content Type" limitation on "Content / Create" policy. And it is reproducible in OE as well.

Checklist:

• Coding standards ($ composer fix-cs)
• Ready for Code Review

[ViniTou]

But I would also like opinion from frontend, just to be sure ;)

Thrown exception is allowedContentTypes.includes is not a function, and its true, in UDW config instead of array of contentTypes there is Object/HashMap with numerical keys. Thing is where we should be consistent about types, in php those two thins (array - hashMap) are the same thing. Should we always reindex before returning to frontend or maybe you should try to always cast them to array. Or just to be sure, both as we can not relay on some common contract?
@dew326

[dew326]

I think it depends what frontend expects, we are not able to make some general rule.
Here is a list of properties with types required by UDW: https://github.com/ezsystems/ezplatform-admin-ui/blob/master/src/bundle/ui-dev/src/modules/universal-discovery/universal.discovery.module.js#L364

[tomaszszopinski]

QA approved on eZPlatform ee 3.0.5 with diff.

[lserwatka]

@ViniTou could you merge it up?