Remove br from save and add to presentation

iszmais · Jul 2, 2024 · 0ae0ca8 · 0ae0ca8
1 parent cc62bdd
commit 0ae0ca8
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 3 deletions.
diff --git a/components/ILIAS/DataCollection/classes/Content/class.ilDclRecordListGUI.php b/components/ILIAS/DataCollection/classes/Content/class.ilDclRecordListGUI.php
@@ -195,7 +195,7 @@ public function listRecords(bool $use_tableview_filter = false): void
             if ($ilSetting->get('advanced_editing_javascript_editor')) {
                 $desc = "<div class='ilDclTableDescription'>" . $desc . "</div>";
             } else {
-                $desc = "<div class='ilDclTableDescription'>" . nl2br(ilUtil::stripSlashes($desc)) . "</div>";
+                $desc = "<div class='ilDclTableDescription'>" .  nl2br($this->refinery->encode()->htmlSpecialCharsAsEntities()->transform($desc)) . "</div>";
             }
         }
         $this->tpl->setContent($desc . $list->getHTML());

diff --git a/components/ILIAS/DataCollection/classes/Fields/Text/class.ilDclTextRecordFieldModel.php b/components/ILIAS/DataCollection/classes/Fields/Text/class.ilDclTextRecordFieldModel.php
@@ -133,7 +133,7 @@ public function parseValue($value)
         if ($this->getField()->getProperty(ilDclBaseFieldModel::PROP_TEXTAREA)
             && !$this->getField()->getProperty(ilDclBaseFieldModel::PROP_URL)
         ) {
-            return nl2br($value);
+            return $value;
         }
 
         return $value;

diff --git a/components/ILIAS/DataCollection/classes/Fields/Text/class.ilDclTextRecordRepresentation.php b/components/ILIAS/DataCollection/classes/Fields/Text/class.ilDclTextRecordRepresentation.php
@@ -69,7 +69,7 @@ public function getHTML(bool $link = true, array $options = []): string
                 'renderRecord'
             ) . '">' . $value . '</a>';
         } else {
-            $html = (is_array($value) && isset($value['link'])) ? $value['link'] : $value;
+            $html = (is_array($value) && isset($value['link'])) ? $value['link'] : nl2br($value);
         }
 
         if (!$html) {

diff --git a/components/ILIAS/DataCollection/classes/Setup/class.ilDataCollectionDBUpdateSteps9.php b/components/ILIAS/DataCollection/classes/Setup/class.ilDataCollectionDBUpdateSteps9.php
@@ -165,4 +165,16 @@ public function step_10(): void
             $this->db->addIndex('il_dcl_tfield_set', array('table_id'), 'i4');
         }
     }
+
+    public function step_11(): void
+    {
+        $this->db->manipulateF(
+            'UPDATE il_dcl_stloc1_value v '.
+            'INNER JOIN il_dcl_record_field rf ON rf.id = v.record_field_id ' .
+            'INNER JOIN il_dcl_field f ON f.id = rf.field_id ' .
+            'SET v.value = REPLACE(v.value, "<br />", "\r\n") WHERE f.datatype_id = %s',
+            [ilDBConstants::T_INTEGER],
+            [ilDclDatatype::INPUTFORMAT_TEXT]
+        );
+    }
 }