You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now collective.cover depends on plone.app.relationfield which can easily made optional. This (unused?) dependency brings more problems than benefits and can be added independently if needed.
Problem is that plone.app.relationfield is dependent of plone.app.intid which, on install, loops through the whole catalog and does a getObject on each brain:
... this operation will never succeed if you have a medium or large database already, with ten of thousand of objects/brains for example and you will never be able to install the collective.cover.
Removing dependency of plone.app.relationfield can be done easily by removing the followings:
a PR is highly appreciated; plaase document what to do on upgrades to not break current installations, the same we did with other hard dependencies we have been removing in the last months.
Right now collective.cover depends on plone.app.relationfield which can easily made optional. This (unused?) dependency brings more problems than benefits and can be added independently if needed.
Problem is that plone.app.relationfield is dependent of plone.app.intid which, on install, loops through the whole catalog and does a getObject on each brain:
... this operation will never succeed if you have a medium or large database already, with ten of thousand of objects/brains for example and you will never be able to install the collective.cover.
Removing dependency of plone.app.relationfield can be done easily by removing the followings:
The text was updated successfully, but these errors were encountered: