Merge branch 'master' into fixes/649

TYPO3-Headless · Oct 2, 2023 · 31cba47 · 31cba47
2 parents a0ae083 + 7d5a90e
commit 31cba47
Show file tree

Hide file tree

Showing 18 changed files with 104 additions and 68 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/Classes/Event/Listener/AfterCacheableContentIsGeneratedListener.php b/Classes/Event/Listener/AfterCacheableContentIsGeneratedListener.php
@@ -20,9 +20,7 @@
 
 class AfterCacheableContentIsGeneratedListener
 {
-    public function __construct(private readonly JsonEncoder $encoder)
-    {
-    }
+    public function __construct(private readonly JsonEncoder $encoder) {}
 
     public function __invoke(AfterCacheableContentIsGeneratedEvent $event)
     {

diff --git a/Classes/Event/Listener/AfterLinkIsGeneratedListener.php b/Classes/Event/Listener/AfterLinkIsGeneratedListener.php
@@ -21,8 +21,7 @@ final class AfterLinkIsGeneratedListener
     public function __construct(
         private readonly HeadlessFrontendUrlInterface $urlUtility,
         private readonly LinkService $linkService
-    ) {
-    }
+    ) {}
 
     public function __invoke(AfterLinkIsGeneratedEvent $event): void
     {

diff --git a/Classes/Event/Listener/AfterPagePreviewUriGeneratedListener.php b/Classes/Event/Listener/AfterPagePreviewUriGeneratedListener.php
@@ -17,9 +17,7 @@
 
 final class AfterPagePreviewUriGeneratedListener
 {
-    public function __construct(private readonly HeadlessFrontendUrlInterface $urlUtility)
-    {
-    }
+    public function __construct(private readonly HeadlessFrontendUrlInterface $urlUtility) {}
 
     public function __invoke(AfterPagePreviewUriGeneratedEvent $event): void
     {

diff --git a/Classes/Form/Decorator/FormDefinitionDecorator.php b/Classes/Form/Decorator/FormDefinitionDecorator.php
@@ -11,6 +11,4 @@
 
 namespace FriendsOfTYPO3\Headless\Form\Decorator;
 
-class FormDefinitionDecorator extends AbstractFormDefinitionDecorator
-{
-}
+class FormDefinitionDecorator extends AbstractFormDefinitionDecorator {}
diff --git a/Classes/ViewHelpers/DomainViewHelper.php b/Classes/ViewHelpers/DomainViewHelper.php
@@ -38,7 +38,7 @@ public static function renderStatic(
             case 'frontendBase':
                 return $urlUtility->getFrontendUrl();
             case 'proxyUrl':
-                return  $urlUtility->getProxyUrl();
+                return $urlUtility->getProxyUrl();
             case 'storageProxyUrl':
                 return $urlUtility->getStorageProxyUrl();
         }

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 {
                             tt_content.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
@@ -10,7 +10,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
@@ -10,7 +10,7 @@ tt_content.textpic {
                 bodytext = TEXT
                 bodytext {
                     field = bodytext
-                    parseFunc =< lib.parseFunc_links
+                    parseFunc =< lib.parseFunc_RTE
                 }
                 gallery = TEXT
                 gallery {

diff --git a/...cript/ContentElement/ParseFunc.typoscript → ...n/TypoScript/Helpers/ParseFunc.typoscript b/...cript/ContentElement/ParseFunc.typoscript → ...n/TypoScript/Helpers/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 {
@@ -125,8 +143,10 @@ lib {
                     parameter.data = parameters : allParams
                     extTarget.data = parameters:target
                 }
+
                 parseFunc.constants = 1
             }
+
             a = TEXT
             a {
                 current = 1

diff --git a/Configuration/TypoScript/Helpers/RenderChildren.typoscript b/Configuration/TypoScript/Helpers/RenderChildren.typoscript
@@ -0,0 +1,44 @@
+/**
+* This lib renders children in headless.
+* To use this code copy code below and replace FIELDNAME and TABLENAME with correct values for relation
+*
+children =< lib.renderChildren
+children.10 {
+  source.field = FIELDNAME
+  tables = TABLENAME
+  conf.TABLENAME.stdWrap.wrap = |###BREAK###
+}
+*
+*/
+
+lib.renderChildren = COA
+lib.renderChildren {
+    10 = RECORDS
+    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/setup.typoscript b/Configuration/TypoScript/setup.typoscript
@@ -6,6 +6,8 @@ plugin.tx_headless {
 
 ## Include page
 @import "EXT:headless/Configuration/TypoScript/Page/*.typoscript"
+## Include helpers
+@import "EXT:headless/Configuration/TypoScript/Helpers/*.typoscript"
 ## Include content elements
 @import "EXT:headless/Configuration/TypoScript/ContentElement/*.typoscript"
 ## Include configuration

diff --git a/Documentation/Developer/Index.rst b/Documentation/Developer/Index.rst
@@ -297,7 +297,7 @@ object with a header definition `lib.contentElementWithHeader`:
           bodytext = TEXT
           bodytext {
             field = bodytext
-            parseFunc =< lib.parseFunc_links
+            parseFunc =< lib.parseFunc_RTE
           }
           demoSubfields {
             fields {

diff --git a/Tests/Unit/ContentObject/DataProcessingExample.php b/Tests/Unit/ContentObject/DataProcessingExample.php
@@ -34,6 +34,6 @@ public function process(
 
         $processedData[$targetVariableName] = ['SomeCustomProcessing'];
 
-        return  $processedData;
+        return $processedData;
     }
 }
diff --git a/Tests/Unit/ContentObject/EmptyDataProcessingExample.php b/Tests/Unit/ContentObject/EmptyDataProcessingExample.php
@@ -33,6 +33,6 @@ public function process(
 
         $processedData[$targetVariableName] = [null];
 
-        return  $processedData;
+        return $processedData;
     }
 }
diff --git a/Tests/Unit/DataProcessing/RootSiteProcessing/TestSiteProvider.php b/Tests/Unit/DataProcessing/RootSiteProcessing/TestSiteProvider.php
@@ -16,9 +16,7 @@
 
 class TestSiteProvider implements SiteProviderInterface
 {
-    public function prepare(array $config, int $siteUid)
-    {
-    }
+    public function prepare(array $config, int $siteUid) {}
 
     public function getSites(): array
     {

diff --git a/composer.json b/composer.json
@@ -30,7 +30,7 @@
     "ext-json": "*",
     "typo3/cms-core": "^12.4",
     "typo3/cms-install": "^12.4",
-    "typo3/coding-standards": "^0.7.1"
+    "typo3/coding-standards": "dev-main"
   },
   "require-dev": {
     "ergebnis/composer-normalize": "^2.15.0",