wcm-io / io.wcm.caconfig.editor

Context-Aware Configuration Editor Template for AEM.
Apache License 2.0
3 stars 7 forks source link

Ca config collections - showing as outdated reference when quick publishing #48

Open srikanthgurram-17 opened 1 week ago

srikanthgurram-17 commented 1 week ago

Hi @stefanseifert GoodDay

We are using 1.16.6 version of ca config editor where we have multiple ca configurations where each config has multiple fields for eg: background color config has color name, color value, checkbox field.

Steps to reproduce the problem:

  1. Open a existing background color configuration (this config already exists for 10 different colors)
  2. Edit one config inside the above background color configuration
  3. save and try quick publish
  4. page redirects to another screen where we see list of outdated configs Expected result: I should see only the config that I edited in the outdated config list. and I should be easily able to publish single config.

Actual result: Instead I see all the 10 background color configs listed as outdated. this is a problem if you have more number configs for eg: around 200 items . this would take lot of time to quick publish and resulting to 503 sometimes

Thanks in advance!

stefanseifert commented 3 days ago

i assume you are using the wcm.io AEM Page persistence strategy from https://wcm.io/caconfig/extensions which stores each list item in a separate page, which should theoretically only show modified/new list items when publishing again.

actually, the problem happens as you are describing: if only one list item is updated, all list items and the list page itself is showed as outdated. the reason is, that on each save all list items are re-saved, and the jcr:lastModified property of all items and the list itself are updated as well, so all are offered for re-publication.

the persistence strategy should be adapted in a way, that jcr:lastModified is only updated when actual changes are done when saving the list with it's items. i will have a look on it, but cannot promise a date for a fix yet (feel free to open a PR to get it done faster).

srikanthgurram-17 commented 3 hours ago

I will come up with a PR for this!