Reference index not updated about t3x-rte_ckeditor_image

[prdt3e]

we should find a universal solution.
That would be an impertinence for every field and for every extension to add.
@mbrodala what do you mean?

[muh-nr]

Hi @mbrodala,
thanks for your hint.
Not simple to count it down again after removing an image.

Feel free to add a pull request.

[mbrodala]

In fact we have ported the soft reference index from sgalinski/tinymce4-rte and it's working just fine. Adding images (even resized) increases the reference count, removing them again decreases it. The DataHandler takes care of all this but it needs a hint, how many references are in a blob of text.

We'll test this a little longer and if it works fine, I'll add a PR with this. :-)

[muh-nr]

Sounds good. Would be very nice.

[muh-nr]

@mbrodala,
any news on this?

[mbrodala]

I was off for vacation in the last 2 weeks. Our fix seems to work fine, I just need some time to turn it into a PR. I try to do this this week.

[muh-nr]

Hey @mbrodala, I'm still interested!

[muh-nr]

Hi @mbrodala,
I did some cleanups and merged your changes into master via branch issue-45.
I squashed the commits to not have cleanup commits in master.

Please check the current master.

Thanks for your help!

[mbrodala]

Thanks a lot, looking forward to the next release. :-)

[mbrodala]

For the reference: 8005c8b

[prdt3e]

What if the editor is outside of ['tt_content'] ['columns'] ['bodytext']?
New field? News plugin? Own extension?

[mbrodala]

@prdt3e See the related change, you'll need to add a small TCA overrides snippet for your field.

[prdt3e]

The solution can be found here with a loop.
It would then be all fields in tt_content covered.

https://github.com/FriendsOfTYPO3/rtehtmlarea/blob/master/Configuration/TCA/Overrides/tt_content.php

But I'm not happy.

[sypets]

What is the current status on this?

I tested that a sys_refindex entry gets created when inserting an image in an RTE-field (tt_content.bodytext). This is the case for 9.0.4, but not for 9.0.3 so this looks good.

Is this now fixed in general or does this only work now for some specific fields? (In the patch it looks like the TCA changes are only made for tt_content.bodytext.)

Will updating the reference index also add references for previously inserted images?

[sypets]

Can you please add the "bug" label? This is a bug. If the reference index is not updated, the references will not be shown in the file list which may result in files being removed (by unsuspecting editors) even though they are still being used.

---

Current status: Reference index entry is not created with latest master:

• rte_ckeditor_image: dev-master
• TYPO3 11.5.25 or dev-11.5

---

I noticed that what is returned by RteImageSoftReferenceParser::findImageTags is probably wrong.

return [
            'content'  => implode('', $this->splitContentTags),
            'elements' => $images,
        ];

should be:

return $images;

Comparing what is passed to (the core) SoftReferenceParserResult::create function with an rte image tag, the result is wrapped in an extra content / elements array:

1. with rte image tag

$elements is

content : "<p><img ---"
elements:
  1: 
    matchString":"<img  ..."
    subst: 
     type: "db" 
     recordRef: "sys_file:17804",
     tokenID: "f6f3a30214c2f46e38d28c50e19e65c8"
     tokenValue: "17804"

2. with file link, it looks more like this:

1: 
   matchString":"<a href=  ..."
   subst: 
    type: "db" 
    recordRef: "sys_file:17804",
    tokenID: "f6f3a30214c2f46e38d28c50e19e65c8"
    tokenValue: "file:17804"