rapila / cms-base

The rapila cms’ internals. Please file all issues here if they’re not directly related to a plugin or the sample site.
http://www.rapi.la
3 stars 1 forks source link

Links "external" and "document" in Richtexteditor don't work #261

Open juergmessmer opened 3 years ago

juergmessmer commented 3 years ago

Both "external_link_proxy" and "display_document" don't work, it looks like "id" is always 0. Tested in "master", in both JournalEntryDetail and Page Texttool.

juergmessmer commented 3 years ago

PHP Problem in InputWidgetModule: Most probably my crime at an attempt to optimize! I removed now the extra jsonBaseObjects handling, and now it seems to work.

Requires further testing, and fixing of DocumentInputWidgetModule

juergmessmer commented 3 years ago

Partial fix: external_links with commit b95463587864709f2a3cdd9e511379f58b26fb85

juergmessmer commented 3 years ago

Document_input value is correct when set, when saved in DB: <p><a href="/get_file/display_document/724">Dokument</a></p>

But when output again, with parseStorageForBackendOutput? <p><a href="/get_file/display_document/724%4000cjUY.jpg">Dokument</a></p>

sabberworm commented 3 years ago

I couldn’t find any commit b95463

juergmessmer commented 3 years ago

For some reason the commit link did not work. I looked for solutions. Normally I could just paste the commit string and it would render the link and show the short version ?!

b95463587864709f2a3cdd9e511379f58b26fb85

On 30 Jan 2021, at 09:49, Raphael Schweikert notifications@github.com wrote:

I couldn’t find any commit b95463

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/rapila/cms-base/issues/261#issuecomment-770231949, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAC3NILQJHFW2AYIYRFRZFTS4QTBRANCNFSM4WZHQLIA.