[TASK] Typoscript improvements (#642)

* [TASK] Typoscript improvements - Align latest parseFunc with TYPO3 core changes - change default parsing of rte with correct parseFunc - add helper for handling inline helpers * [TASK] Typoscript improvements - move helpers to dedicated branch - assign example ---------
TYPO3-Headless · Oct 2, 2023 · 6cbfb2c · 6cbfb2c
1 parent 0791bf8
commit 6cbfb2c
Show file tree

Hide file tree

Showing 13 changed files with 137 additions and 86 deletions.
diff --git a/Classes/DataProcessing/DatabaseQueryProcessor.php b/Classes/DataProcessing/DatabaseQueryProcessor.php
@@ -37,7 +37,7 @@
  *      bodytext = TEXT
  *      bodytext {
  *          field = bodytext
- *          parseFunc =< lib.parseFunc_links
+ *          parseFunc =< lib.parseFunc_RTE
  *      }
  *      link = TEXT
  *      link {

diff --git a/Configuration/TypoScript/2.x/ContentElement/Text.typoscript b/Configuration/TypoScript/2.x/ContentElement/Text.typoscript
@@ -6,7 +6,7 @@ tt_content.text {
                 bodytext = TEXT
                 bodytext {
                     field = bodytext
-                    parseFunc =< lib.parseFunc_links
+                    parseFunc =< lib.parseFunc_RTE
                 }
             }
         }

diff --git a/Configuration/TypoScript/2.x/ContentElement/Textmedia.typoscript b/Configuration/TypoScript/2.x/ContentElement/Textmedia.typoscript
@@ -7,7 +7,7 @@ tt_content.textmedia {
                 bodytext = TEXT
                 bodytext {
                     field = bodytext
-                    parseFunc =< lib.parseFunc_links
+                    parseFunc =< lib.parseFunc_RTE
                 }
                 gallery = TEXT
                 gallery {

diff --git a/Configuration/TypoScript/2.x/ContentElement/Textpic.typoscript b/Configuration/TypoScript/2.x/ContentElement/Textpic.typoscript
@@ -7,7 +7,7 @@ tt_content.textpic {
                 bodytext = TEXT
                 bodytext {
                     field = bodytext
-                    parseFunc =< lib.parseFunc_links
+                    parseFunc =< lib.parseFunc_RTE
                 }
                 gallery = TEXT
                 gallery {

diff --git a/Configuration/TypoScript/2.x/Helper/ParseFunc.typoscript b/Configuration/TypoScript/2.x/Helper/ParseFunc.typoscript
@@ -6,37 +6,40 @@ lib {
         makelinks {
             http {
                 keep = {$styles.content.links.keep}
-                extTarget.data = parameters:target
+                extTarget = {$styles.content.links.extTarget}
             }
+
             mailto {
                 keep = path
             }
         }
+
         tags {
-            link = TEXT
-            link {
-                current = 1
-                typolink {
-                    parameter.data = parameters : allParams
-                    extTarget.data = parameters:target
-                }
-                parseFunc.constants = 1
-            }
             a = TEXT
             a {
                 current = 1
                 typolink {
                     parameter.data = parameters:href
                     title.data = parameters:title
                     ATagParams.data = parameters:allParams
-                    target.data = parameters:target
-                    extTarget.data = parameters:target
+                    # the target attribute takes precedence over config.intTarget
+                    target.ifEmpty.data = parameters:target
+                    # the target attribute takes precedence over the constant (styles.content.links.extTarget)
+                    # which takes precedence over config.extTarget
+                    # do not pass extTarget as reference, as it might not be set resulting in the string being
+                    # written to the target attribute
+                    extTarget {
+                        ifEmpty < config.extTarget
+                        ifEmpty.override = {$styles.content.links.extTarget}
+                        override.data = parameters:target
+                    }
                 }
             }
         }
+
         allowTags = {$styles.content.allowTags}
         denyTags = *
-        sword = <span class="ce-sword">|</span>
+        # @deprecated since TYPO3 v12, remove with v13
         constants = 1
         nonTypoTagStdWrap {
             HTMLparser = 1
@@ -51,19 +54,22 @@ lib {
     parseFunc_RTE < lib.parseFunc
     parseFunc_RTE {
         # Processing <ol>, <ul> and <table> blocks separately
-        externalBlocks = article, aside, blockquote, div, dd, dl, footer, header, nav, ol, section, table, ul, pre
+        externalBlocks = article, aside, blockquote, div, dd, dl, footer, header, nav, ol, section, table, ul, pre, figure
         externalBlocks {
             ol {
                 stripNL = 1
-                stdWrap.parseFunc = < lib.parseFunc
+                stdWrap.parseFunc =< lib.parseFunc
             }
+
             ul {
                 stripNL = 1
-                stdWrap.parseFunc = < lib.parseFunc
+                stdWrap.parseFunc =< lib.parseFunc
             }
+
             pre {
                 stdWrap.parseFunc < lib.parseFunc
             }
+
             table {
                 stripNL = 1
                 stdWrap {
@@ -74,25 +80,34 @@ lib {
                             always = 1
                             list = contenttable
                         }
+
                         keepNonMatchedTags = 1
                     }
                 }
+
                 HTMLtableCells = 1
                 HTMLtableCells {
                     # Recursive call to self but without wrapping non-wrapped cell content
                     default.stdWrap {
-                        parseFunc = < lib.parseFunc_RTE
-                        parseFunc.nonTypoTagStdWrap.encapsLines.nonWrappedTag =
+                        parseFunc =< lib.parseFunc_RTE
+                        parseFunc.nonTypoTagStdWrap.encapsLines {
+                            nonWrappedTag =
+                            innerStdWrap_all.ifBlank =
+                        }
                     }
+
                     addChr10BetweenParagraphs = 1
                 }
             }
+
             div {
                 stripNL = 1
                 callRecursive = 1
             }
+
             article < .div
             aside < .div
+            figure < .div
             blockquote < .div
             footer < .div
             header < .div
@@ -101,20 +116,23 @@ lib {
             dl < .div
             dd < .div
         }
+
         nonTypoTagStdWrap {
+            HTMLparser = 1
+            HTMLparser {
+                keepNonMatchedTags = 1
+                htmlSpecialChars = 2
+            }
+
             encapsLines {
                 encapsTagList = p,pre,h1,h2,h3,h4,h5,h6,hr,dt
                 remapTag.DIV = P
                 nonWrappedTag = P
                 innerStdWrap_all.ifBlank = &nbsp;
             }
-            HTMLparser = 1
-            HTMLparser {
-                keepNonMatchedTags = 1
-                htmlSpecialChars = 2
-            }
         }
     }
+
     parseFunc_links {
         htmlSanitize = 1
         tags {
@@ -141,8 +159,3 @@ lib {
         }
     }
 }
-
-
-
-
-
diff --git a/Configuration/TypoScript/ContentElement/Shortcut.typoscript b/Configuration/TypoScript/ContentElement/Shortcut.typoscript
@@ -3,7 +3,7 @@ tt_content.shortcut {
     fields {
         content {
             fields {
-                shortcut = COA
+                shortcut =< lib.renderChildren
                 shortcut {
                     10 = RECORDS
                     10 {
@@ -13,27 +13,6 @@ tt_content.shortcut {
                             {$styles.content.shortcut.tables}.stdWrap.wrap = |###BREAK###
                         }
                     }
-                    stdWrap {
-                        innerWrap = [|]
-                        split {
-                            token = ###BREAK###
-                            cObjNum = 1 |*|2|*| 3
-                            1 {
-                                current = 1
-                                stdWrap.wrap = |
-                            }
-
-                            2 {
-                                current = 1
-                                stdWrap.wrap = ,|
-                            }
-
-                            3 {
-                                current = 1
-                                stdWrap.wrap = |
-                            }
-                        }
-                    }
                 }
             }
         }

diff --git a/Configuration/TypoScript/ContentElement/Text.typoscript b/Configuration/TypoScript/ContentElement/Text.typoscript
@@ -6,7 +6,7 @@ tt_content.text {
                 bodytext = TEXT
                 bodytext {
                     field = bodytext
-                    parseFunc =< lib.parseFunc_links
+                    parseFunc =< lib.parseFunc_RTE
                 }
             }
         }

diff --git a/Configuration/TypoScript/ContentElement/Textmedia.typoscript b/Configuration/TypoScript/ContentElement/Textmedia.typoscript
@@ -7,7 +7,7 @@ tt_content.textmedia {
                 bodytext = TEXT
                 bodytext {
                     field = bodytext
-                    parseFunc =< lib.parseFunc_links
+                    parseFunc =< lib.parseFunc_RTE
                 }
                 gallery = TEXT
                 gallery {

diff --git a/Configuration/TypoScript/ContentElement/Textpic.typoscript b/Configuration/TypoScript/ContentElement/Textpic.typoscript
@@ -7,7 +7,7 @@ tt_content.textpic {
                 bodytext = TEXT
                 bodytext {
                     field = bodytext
-                    parseFunc =< lib.parseFunc_links
+                    parseFunc =< lib.parseFunc_RTE
                 }
                 gallery = TEXT
                 gallery {