Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Open
juergmessmer opened this issue Jan 29, 2021 · 5 comments
Open

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

juergmessmer opened this issue Jan 29, 2021 · 5 comments
Assignees
@sabberworm @juergmessmer
Labels
level-bug logic-js

Comments

@juergmessmer
Copy link
Member

juergmessmer commented Jan 29, 2021 •
edited
Loading

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
Copy link
Member Author

juergmessmer commented Jan 29, 2021 •
edited
Loading

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
Copy link
Member Author

juergmessmer commented Jan 29, 2021 •
edited
Loading

Partial fix: external_links with commit b954635

@juergmessmer
Copy link
Member Author

juergmessmer commented Jan 29, 2021 •
edited
Loading

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
Copy link
Member

I couldn’t find any commit b95463

@juergmessmer
Copy link
Member Author

juergmessmer commented Jan 30, 2021 via email

juergmessmer added a commit that referenced this issue Jan 30, 2021
@juergmessmer
removed double handling of array
b954635 
• an old error due to attempt to optimize. Sorry.
• this fixes the external_link problem in the text editor
  base issue #261
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sabberworm sabberworm

@juergmessmer juergmessmer

Labels
level-bug logic-js
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sabberworm @juergmessmer