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.

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

Review rteEnabled flag #234

Open
fsuter opened this issue Mar 21, 2022 · 3 comments
Open

Review rteEnabled flag #234

fsuter opened this issue Mar 21, 2022 · 3 comments

Comments

@fsuter
Copy link
Contributor

fsuter commented Mar 21, 2022

It is unclear what the rteEnabled flag actually does. This needs to be investigated and - at the very least - better documented.

@ineswillenbrock
Copy link

rteEnabled = false puts the data nearly completely untransformed into the database - nearly: if there are Windows-Newlines (\r\n), they (might?) get converted to Unix-Newlines (\n)

rteEnabled = true seems (!) to use RTE transformations - how to configure them: unclear.

The "default" line break for RTE Stuff after processing is \r\n (CRLF) (https://api.typo3.org/master/class_t_y_p_o3_1_1_c_m_s_1_1_core_1_1_html_1_1_rte_html_parser.html streamlineLineBreaksAfterProcessing());
while processing just \n or LF, see streamlineLineBreaksForProcessing()

Right now I'm trying to understand the way from file to db in external_import - with the hope to find all involved parts and methods.

@ineswillenbrock
Copy link

ineswillenbrock commented Apr 19, 2022

Edit of the previous one: rteEnabled works just fine -- if you don't use the bootstrap_package.

Longer explanation: the default RTE brings its own configuration, that is overwritten if you use the bootstrap_package.
This, the bootstrap_package, configuration seems to have some things that result in the above behaviour.
A possible workaround is the following:
Define the rte configuration in the import configuration like so:

$GLOBALS['TCA']['tx_extname']['columns']['description'] =[
  'config' => [
      'type' => 'text',
     'richtextConfiguration' => 'default_import',
   ],
];

The default_import should be a copy of a non-bootstrap-default.yaml-file.

I'm not sure, WHY this happens, which part of the configuration is responsible for this.

But: rteEnabled=true is fine
edit: will check the configuration and see, if I understand the why.

@fsuter
Copy link
Contributor Author

fsuter commented Apr 26, 2022

Thanks a lot for continuing to dig into that topic! I feel supported and grateful for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants