[CELUM-44] Rich-Text-Links to external DAM do not use pagination / picking remains slow Created: 08/Jun/22  Updated: 13/Jul/22  Resolved: 13/Jul/22

Status: Closed
Project: Celum DAM Connector
Component/s: None
Affects Version/s: None
Fix Version/s: 1.0.3

Type: Bug Priority: Neutral
Reporter: Christopher Chard Assignee: Raphael Falvo
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: Java Source File CelumLinkUtil.java    
Issue Links:
causality
caused by EXTDAM-364 [S3] Performance issue when using Ric... Open
Template:
Acceptance criteria:
Empty
Date of First Response:

 Description   

Pagination-Feature is not used when selecting external dam assets via Rich Text Field. Thus picking assets from external dam (with many assets) is painfully slow.



 Comments   
Comment by Raphael Falvo [ 24/Jun/22 ]

Release 1.0.3-beta9 provides a way to let author choose between standard dam chooser and optimised Celum chooser in RichTextField by mapping components in module descriptor. Documentation describes how to do.

Comment by Raphael Falvo [ 04/Jul/22 ]

Here is the code logic which identifies the asset from the stored link:

https://git.magnolia-cms.com/projects/MODULES/repos/dam/browse/magnolia-dam-app/src/main/java/info/magnolia/dam/app/field/factory/DamRichTextFieldFactory.java#158

Comment by Tobias Szczepanski [ 06/Jul/22 ]

Hi Teresa, since Raphael is on holidays, I guess the two of us can take over

There's a minor issue with the info.magnolia.external.dam.celum.field.factory.MultipleDamsRichTextFieldFactory, i.e. the constructor has default access, therefore I'm not able to extend the class. I need to extend it, as we're extending the RichTextFieldFactory with some custom logic, and with the latest celum-dam-connector we'd use the MultipleDamsRichTextFieldFactory as the superclass.

Best,
Tob

Comment by Teresa Miyar [ 13/Jul/22 ]

Provided a CelumLinkUtil the solution can be improved by iterating the providers, this was just a quick fix.

 

Generated at Sun Feb 11 23:58:30 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.