plone / plone.batching

Batching facilities for Plone
https://pypi.org/project/plone.batching
2 stars 8 forks source link

Use context.absolute_url instead request.ACTUAL_URL for generating batch links #13

Closed thet closed 8 years ago

thet commented 8 years ago

For generating the batching urls, don't use ACTUAL_URL, but the context's absolute_url. This allows for mutliple batchnavigations on different contexts shown on one page via some collage or mosaic view.

ATTENTION: CAN'T BE DONE THAT WAY!

PROBLEM: On views, which are no defaultView but explicitly called (e.g. @@folder_contents), the links would be generated wrong. Couldn't determine the actual shown view within one request cycle, so I gave up on this. It's just a corne use case anyways...

This pull request is just for documentation. I'll close it right away.

thet commented 8 years ago

@spereverde had a use case, where she showed multiple collections with batch navigation on a full view (see: https://github.com/tomgross/plone.batching/pull/1 ). Actually, maybe this idea should be followed on...