function pwg_nl2br useless ? #2191

New Issue

2026-10-05T01:09:12+02:00

adam commented

2026-10-05 01:09:12 +02:00

In regard of: https://fr.piwigo.org/forum/viewtopic.php?pid=243798#p243798

In regard of: https://fr.piwigo.org/forum/viewtopic.php?pid=243798#p243798

adam commented

2026-10-05 01:09:13 +02:00

This looks promising in my opinion :

  // Check if content already contains HTML tags
  // Match opening tags like <p>, <br>, <div>, etc.
  if (preg_match('/<[a-z][\w-]*(?:\s[^>]*)?>/i', $string))
  {
    // Content has HTML, don't add <br>
    return $string;
  }

This looks promising in my opinion : ``` // Check if content already contains HTML tags // Match opening tags like , , <div>, etc. if (preg_match('/<[a-z][\w-]*(?:\s[^>]*)?>/i', $string)) { // Content has HTML, don't add return $string; } ```

adam commented

2026-10-05 01:09:13 +02:00

Forget, it's AI bullshit.

Forget, it's AI bullshit.

adam commented

2026-10-05 01:09:14 +02:00

I don't care. I don't reject AI "by default". The useless addition of   exists. I think detecting HTML is a good solution. Period.

I don't care. I don't reject AI "by default". The useless addition of ` ` exists. I think detecting HTML is a good solution. Period.

Sign in to join this conversation.

master

issue-2560-people

translation

16.x

feat/embedding-based-contextual-tagging

issue-2576-search-include-ai-in-search-results

feat/search-ocr-ai

issue-2391-update-menu-design

15.x

detect-container

issue-2386-update-filters-users-activity

issue-2362-add-filters-options-in-admin

issue-2367-imagemagick7

issue-2225-quicksearch-help

14.x

issue-2164-BatchManagerUnitMode-Redesign

13.x

12.x

issue-1657-SVG_Support

issue-1703-historySearchMethod

11.x

2.10

2.9

2.8

2.7

2.5

2.6

2.4

2.3

2.2

2.1

2.0

1.7

1.3

1.6

1.5

1.4

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: adam/Piwigo#2191

function pwg_nl2br useless ? #2191

function `pwg_nl2br` useless ? #2191