search

enonic / app-contentstudio

Content Studio of Enonic XP
GNU General Public License v3.0
10 stars 4 forks source link

Changing content permissions triggers fetch of all modified child content items #1285

Closed GlennRicaud closed 4 years ago

GlennRicaud commented 4 years ago

Calls to resolveByIds should be batched (But check first it is really necessary to retrieve all of them).

Example: Change permissions on the parent of a lot of contents ContentStudio will call "resolveByIds" with the list of all modified contents

Screenshot 2019-12-04 at 16 14 34

Fix also in 2.2.4

Check for similar situations

sigdestad commented 4 years ago

We also need a circut breaker for this stuff in XP! Can never trust a client!

GlennRicaud commented 4 years ago

We also need a circut breaker for this stuff in XP! Can never trust a client!

https://github.com/enonic/xp/issues/7693

runarmyklebust commented 4 years ago

This will be always be a problem if running with several content studios open and someone doing changes to a lot of content?

ashklianko commented 4 years ago

Considering UI part of task: I'll optimize those calls on UI so we won't fetch all ids, only needed ones