Compare commits

..
4 Commits
Author SHA1 Message Date
kantarciandClaude Opus 5 741eaa80f7 Release 1.21.0
CMS quality / php (8.2) (push) Canceled after 0s
CMS quality / php (8.4) (push) Canceled after 0s
CMS quality / browser (push) Canceled after 0s
CMS quality / mysql (push) Canceled after 0s
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-16 09:18:34 +02:00
kantarciandClaude Opus 5 23fdca4edb Move image corners into the design area
Umut on the DEEG hero image: "It should be rounded, but it was done from
somewhere, and the corners look odd. And the editor has no image corner
rounding at all, does it?"

It did have one - per block, under Design > Bild > Bildecken. What it did not
have was the 12px that enhancements.css set on .block-image for every
installation: rounded on screen, present in no field, and only removable by
overriding it on each block. On a template that is square throughout it shows,
because the elements above the image - the gradient, the sign, the blue call
to action - are square and stick out of the rounded corner.

The radius is now theme.base.imageRadius, validated like the container width,
emitted as --k-image-radius and used by the stylesheet. The default stays 12px
so existing sites look the same; 0px makes every image square. The per-block
setting still wins.

DEEG is set to 0px, in the seed script and in the installation.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-16 09:18:27 +02:00
kantarciandClaude Opus 5 a8368068aa Release 1.20.2
CMS quality / php (8.2) (push) Canceled after 0s
CMS quality / php (8.4) (push) Canceled after 0s
CMS quality / browser (push) Canceled after 0s
CMS quality / mysql (push) Canceled after 0s
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-16 09:05:17 +02:00
kantarciandClaude Opus 5 f4ec0d9877 Make the entity repair a console command, not a script
scripts/ is on the exclusion list of build-release.php, so an installation
that runs from the release archive - the usual case on shared hosting -
would never have seen scripts/repair-doubled-entities.php. The 1.20.1 notes
told everyone to run it, which was only true for a Git working copy.

cms:repair:entities does the same work and ships with the application: a dry
run by default, --schreiben to apply, each change validated the way saving
validates it (pages with the heading rule, areas without, as in
PartialRepository). The script is gone; there is one way to do this now.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-16 09:05:11 +02:00
11 changed files with 320 additions and 134 deletions

No files matched your search

+5 -1
View File
@@ -4,9 +4,13 @@ Selbst betreibbares Block-CMS für PHP-Webhosting mit visuellem Editor nach dem
Die übernommenen Originaldokumente sind `AGENTS.md` und `ENTWICKLUNGSPLAN.md`. Den tatsächlichen Stand enthält `docs/STATUS.md`, Entscheidungen und Datenmodell stehen in `docs/ARCHITECTURE.md`.
## Bildecken unter Design (ab 1.21.0)
Wie stark die Ecken der Bilder gerundet sind, steht jetzt unter **Design → Bildecken** und gilt für die ganze Website – Bilder, Galerien, Logoleisten und Beitragslisten. Vorgabe ist `12px` wie bisher, `0px` macht sie eckig. Ein einzelnes Bild lässt sich davon abweichend einstellen: im Editor beim ausgewählten Bild unter **Design → Bild → Bildecken**, dort auch je Gerät und je Ecke. Vorher stand die Rundung fest im Stylesheet und war in keinem Feld zu finden.
## Kaufmännisches Und und Zeilenumbrüche bleiben, wie sie sind (ab 1.20.1)
**Behoben.** Ein Text ohne Auszeichnung wurde beim Speichern doppelt kodiert: aus `&` wurde `&amp;`, beim nächsten Speichern `&amp;amp;` – irgendwann stand die Entity sichtbar auf der Seite. Ebenso wuchsen Zeilenumbrüche: jedes zweite Speichern machte aus einem `<br>` zwei. Beides ist behoben, und wiederholtes Speichern ändert einen Text jetzt nicht mehr. Was bereits falsch gespeichert ist, räumt `php scripts/repair-doubled-entities.php` auf (erst Trockenlauf, dann `--schreiben`).
**Behoben.** Ein Text ohne Auszeichnung wurde beim Speichern doppelt kodiert: aus `&` wurde `&amp;`, beim nächsten Speichern `&amp;amp;` – irgendwann stand die Entity sichtbar auf der Seite. Ebenso wuchsen Zeilenumbrüche: jedes zweite Speichern machte aus einem `<br>` zwei. Beides ist behoben, und wiederholtes Speichern ändert einen Text jetzt nicht mehr. Was bereits falsch gespeichert ist, räumt `php bin/console cms:repair:entities` auf: ohne Option zeigt er nur, was sich ändern würde, mit `--schreiben` wendet er es an. Angefasst werden die veröffentlichte Fassung und der Entwurf jeder Seite sowie die Bereiche; ältere Fassungen bleiben Historie.
## Eigenes HTML und eingebettete Seiten (ab 1.20.0)
+1 -1
View File
@@ -1 +1 @@
1.20.1
1.21.0
+14 -2
View File
@@ -1,5 +1,17 @@
# Projektstatus
## Fassung 1.21.0: Bildecken gehören dem Designbereich (16.09.2026)
**Anlass:** Umut zum Hero-Bild von DEEG: „Das sollte abgerundet sein, aber wurde von irgendwo gemacht, und die Ecken sehen komisch aus. Es gibt auch im Editor gar nicht Bildecken abrunden, oder?“
**Was los war.** `.block-image{border-radius:12px}` stand fest in `enhancements.css` – jedes Bild jeder Installation war gerundet, ohne dass ein Feld das gesagt hätte. Auf DEEG (eine durchgehend eckige Vorlage) fiel es auf, weil über dem gerundeten Bild eckige Elemente liegen: der dunkle Verlauf, das Schild, die blaue Aufruf-Fläche. An den Ecken schaute deshalb die eckige Fläche aus der Rundung heraus.
**Behoben.** Die Ecken sind jetzt eine Einstellung unter **Design → Bildecken** (`theme.base.imageRadius`, geprüft wie die Inhaltsbreite, ausgegeben als `--k-image-radius`). Vorgabe bleibt `12px`, damit bestehende Websites gleich aussehen; `0px` macht alle Bilder eckig. Für ein einzelnes Bild gab es die Einstellung schon immer – im Editor unter **Design → Bild → Bildecken** (`imageRadius` im Stilschema, für Bild, Galerie, Logoleiste und Beitragsliste); sie überschreibt den Wert der Website.
**DEEG:** `imageRadius` auf `0px`, im Aufbauskript und in der Installation. Das Hero-Bild ist damit eckig wie die Vorlage, und die Ecken stimmen wieder.
**Geprüft:** neuer Test in `tests/CmsTest.php`: Vorgabe ist `12px`, eine Seite mit Bild trägt `--k-image-radius:12px`, nach dem Speichern von `0px` trägt sie `--k-image-radius:0px`, und „rund“ wird abgewiesen wie jede andere Nicht-Größenangabe.
## Fassung 1.20.1: das kaufmännische Und, das bei jedem Speichern wuchs (16.09.2026)
**Anlass:** Umut: „Beim Startseitenbild wird der Text & umgewandelt, aber in der Ansicht nach dem Speichern wird &amp; daraus.“
@@ -8,9 +20,9 @@
**Behoben.** Der Zweig dekodiert jetzt zuerst (`html_entity_decode`) und escaped danach, und er ersetzt den Umbruch, statt ihn zu ergänzen. Damit endet er bei derselben Darstellung wie der DOM-Zweig, und beide sind unveränderlich gegenüber wiederholtem Speichern.
**Bestand.** `scripts/repair-doubled-entities.php` räumt auf, was schon gespeichert ist: veröffentlichte Fassung und Entwurf jeder Seite sowie die Bereiche, mit Trockenlauf und `--schreiben`, jede Änderung gegen den Validator geprüft. Ältere Fassungen bleiben Historie. Auf DEEG angewandt (eine Stelle im Entwurf der Startseite). **Offen: kant-it.de** – dort stehen zwei dreifach kodierte Stellen im Entwurf der Startseite; das Skript gehört nach dem Update dort einmal ausgeführt, sonst steht der Murks beim nächsten Veröffentlichen live.
**Bestand.** Der Konsolenbefehl `bin/console cms:repair:entities` räumt auf, was schon gespeichert ist: veröffentlichte Fassung und Entwurf jeder Seite sowie die Bereiche, mit Trockenlauf und `--schreiben`, jede Änderung gegen den Validator geprüft (Seiten wie beim Speichern, Bereiche ohne Überschriftspflicht). Als Befehl und nicht als Skript unter `scripts/`, weil dieser Ordner nicht ins Releasearchiv geht – eine Installation ohne Git hätte das Skript sonst nie gesehen. Ältere Fassungen bleiben Historie. Auf DEEG angewandt (eine Stelle im Entwurf der Startseite). **Offen: kant-it.de** – dort stehen zwei dreifach kodierte Stellen im Entwurf der Startseite; das Skript gehört nach dem Update dort einmal ausgeführt, sonst steht der Murks beim nächsten Veröffentlichen live.
**Geprüft:** neuer Test in `tests/CmsTest.php`: sechs Eingaben (Entity, nacktes Und, `<`, Zeilenumbruch, Markup, wörtliche Tags) dreimal hintereinander durch `sanitise()` – jedes Mal dasselbe Ergebnis; dazu eine Seite, die dreimal gespeichert und veröffentlicht wird und danach immer noch ein einziges `&amp;` trägt.
**Geprüft:** `tests/RepairEntitiesCommandTest.php` (Trockenlauf ändert nichts; Schreiben entschachtelt drei Stufen und lässt ein richtiges `&amp;` stehen; ein zweiter Lauf findet nichts mehr) und ein neuer Test in `tests/CmsTest.php`: sechs Eingaben (Entity, nacktes Und, `<`, Zeilenumbruch, Markup, wörtliche Tags) dreimal hintereinander durch `sanitise()` – jedes Mal dasselbe Ergebnis; dazu eine Seite, die dreimal gespeichert und veröffentlicht wird und danach immer noch ein einziges `&amp;` trägt.
**Nebenbei (DEEG):** Das Favicon zeigte den mittigen Ausschnitt des Logos, also „EE“ aus dem Schriftzug. Neu ist ein eigenes quadratisches Zeichen: der Dachgiebel aus dem Logo, weiß auf Logo-Blau, gerendert aus `logo-deeg.png` von `scripts/set-immobilien-deeg-favicon.php`. Vier Entwürfe standen zur Wahl (Doppeldach, Giebel, Dach blau auf weiß, Buchstabe D), Umut hat den Giebel gewählt.
+1 -1
View File
@@ -1,4 +1,4 @@
.block-gallery{display:grid;grid-template-columns:repeat(3,minmax(0,1fr));gap:24px;margin-block:24px}.block-gallery>.block-image{min-width:0;margin:0}.block-gallery img{width:100%;aspect-ratio:4/3;object-fit:cover}.block-image figcaption{font-size:14px;line-height:1.5;padding:12px 0;color:inherit}.fx-shadow-none{box-shadow:none}.fx-shadow-soft{box-shadow:0 8px 24px #142e251f}.fx-shadow-strong{box-shadow:0 16px 40px #142e2540}.block-image{border-radius:12px}.block-image img{border-radius:inherit}.block-gallery:empty{min-height:32px}.fx-hover-zoom{overflow:hidden}.fx-hover-zoom img{transform-origin:center}.tone-ink figure{color:var(--paper)}
.block-gallery{display:grid;grid-template-columns:repeat(3,minmax(0,1fr));gap:24px;margin-block:24px}.block-gallery>.block-image{min-width:0;margin:0}.block-gallery img{width:100%;aspect-ratio:4/3;object-fit:cover}.block-image figcaption{font-size:14px;line-height:1.5;padding:12px 0;color:inherit}.fx-shadow-none{box-shadow:none}.fx-shadow-soft{box-shadow:0 8px 24px #142e251f}.fx-shadow-strong{box-shadow:0 16px 40px #142e2540}.block-image{border-radius:var(--k-image-radius,12px)}.block-image img{border-radius:inherit}.block-gallery:empty{min-height:32px}.fx-hover-zoom{overflow:hidden}.fx-hover-zoom img{transform-origin:center}.tone-ink figure{color:var(--paper)}
@media(max-width:1024px){.block-gallery{grid-template-columns:repeat(2,minmax(0,1fr))}}
@media(max-width:767px){.block-gallery{grid-template-columns:minmax(0,1fr);gap:16px}}
.fx-hover-lift,.fx-hover-zoom img{transition:transform .25s ease}.fx-hover-lift:hover,.fx-hover-lift:focus-within{transform:translateY(-4px)}.fx-hover-zoom:hover img,.fx-hover-zoom:focus-within img{transform:scale(1.035)}
-127
View File
@@ -1,127 +0,0 @@
<?php
/**
* Repariert Texte, die durch das doppelte Kodieren in `RichText::sanitise()` verschachtelt wurden.
*
* Bis 1.20.0 hat der Zweig für Text ohne Tags die Eingabe nur escaped, statt sie vorher zu
* dekodieren. Aus „Freudenstadt &amp; Umgebung“ wurde bei jedem Speichern eine Stufe mehr
* (`&amp;amp;`, `&amp;amp;amp;` …), bis die Entity sichtbar auf der Seite stand. Der Fehler ist
* behoben; dieses Skript räumt auf, was schon gespeichert ist.
*
* Angefasst werden die Dokumente, die tatsächlich in Gebrauch sind: die veröffentlichte und die
* Entwurfsfassung jeder Seite sowie die Bereiche. Ältere Fassungen bleiben, wie sie sind – sie
* sind Historie; wer eine davon zurückholt, speichert sie danach einmal, und der reparierte Kern
* schreibt sie richtig.
*
* Aufruf: php scripts/repair-doubled-entities.php [--schreiben]
* Ohne `--schreiben` wird nur gezeigt, was sich ändern würde.
*/
require dirname(__DIR__).'/config/bootstrap.php';
use App\Content\DocumentValidator;
use App\Infrastructure\ConnectionFactory;
$root = dirname(__DIR__);
$schreiben = in_array('--schreiben', $argv, true);
$url = str_replace('%kernel.project_dir%', str_replace('\\', '/', $root), (string) $_SERVER['DATABASE_URL']);
$db = ConnectionFactory::create($url);
$validator = new DocumentValidator();
/** Entschachtelt eine doppelt kodierte Entity, so oft wie nötig: &amp;amp;amp; wird zu &amp;. */
$entschachteln = static function (string $text): string {
$vorher = null;
// Nur eine Entity hinter einem &amp; ist ein Fehler - ein einzelnes &amp; ist richtig.
while ($vorher !== $text) {
$vorher = $text;
$text = preg_replace('/&amp;(amp|lt|gt|quot|apos|nbsp|#\d{1,6}|#x[0-9a-fA-F]{1,6});/', '&$1;', $text) ?? $text;
}
return $text;
};
/** Läuft durch die Bausteine und korrigiert die Texte; zählt die Änderungen. */
$bausteine = function (array $liste, int &$zahl, array &$beispiele) use (&$bausteine, $entschachteln): array {
foreach ($liste as $index => $block) {
foreach (['text', 'caption', 'label', 'title', 'hint', 'provider'] as $feld) {
$wert = $block['props'][$feld] ?? null;
if (!is_string($wert) || '' === $wert) {
continue;
}
$neu = $entschachteln($wert);
if ($neu !== $wert) {
$liste[$index]['props'][$feld] = $neu;
++$zahl;
if (count($beispiele) < 6) {
$beispiele[] = mb_substr($wert, 0, 60).' → '.mb_substr($neu, 0, 60);
}
}
}
if (isset($block['children']) && is_array($block['children'])) {
$liste[$index]['children'] = $bausteine($block['children'], $zahl, $beispiele);
}
}
return $liste;
};
$gesamt = 0;
$berichte = [];
// Die aktiven Fassungen jeder Seite: veröffentlicht und Entwurf.
$aktive = [];
foreach ($db->fetchAllAssociative('SELECT id, path, published_revision_id, draft_revision_id FROM page') as $seite) {
foreach (['published_revision_id' => 'veröffentlicht', 'draft_revision_id' => 'Entwurf'] as $spalte => $was) {
if (!empty($seite[$spalte])) {
$aktive[(string) $seite[$spalte]] = '/'.$seite['path'].' ('.$was.')';
}
}
}
foreach ($aktive as $revision => $bezeichnung) {
$roh = $db->fetchOne('SELECT document FROM revision WHERE id = ?', [$revision]);
if (!is_string($roh)) {
continue;
}
$dokument = json_decode($roh, true, 512, JSON_THROW_ON_ERROR);
$zahl = 0;
$beispiele = [];
$dokument['blocks'] = $bausteine($dokument['blocks'] ?? [], $zahl, $beispiele);
if (0 === $zahl) {
continue;
}
$gesamt += $zahl;
$berichte[] = ['was' => $bezeichnung, 'zahl' => $zahl, 'beispiele' => $beispiele];
if ($schreiben) {
// Geprüft wird wie beim Speichern - eine Korrektur darf kein ungültiges Dokument hinterlassen.
$validator->validate($dokument, writing: true);
$db->executeStatement('UPDATE revision SET document = ? WHERE id = ?', [json_encode($dokument, JSON_THROW_ON_ERROR | JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES), $revision]);
}
}
// Und die Bereiche (Kopf, Fuß, verknüpfte Sektionen).
foreach ($db->fetchAllAssociative('SELECT id, kind, name, document FROM partial') as $bereich) {
$dokument = json_decode((string) $bereich['document'], true, 512, JSON_THROW_ON_ERROR);
$zahl = 0;
$beispiele = [];
$dokument['blocks'] = $bausteine($dokument['blocks'] ?? [], $zahl, $beispiele);
if (0 === $zahl) {
continue;
}
$gesamt += $zahl;
$berichte[] = ['was' => 'Bereich '.$bereich['kind'].' „'.$bereich['name'].'“', 'zahl' => $zahl, 'beispiele' => $beispiele];
if ($schreiben) {
$validator->validate($dokument, writing: true);
$db->executeStatement('UPDATE partial SET document = ? WHERE id = ?', [json_encode($dokument, JSON_THROW_ON_ERROR | JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES), $bereich['id']]);
}
}
foreach ($berichte as $bericht) {
echo $bericht['was'], ': ', $bericht['zahl'], ' Stelle(n)', "\n";
foreach ($bericht['beispiele'] as $beispiel) {
echo ' ', $beispiel, "\n";
}
}
if (0 === $gesamt) {
echo "Nichts zu reparieren.\n";
exit(0);
}
echo $schreiben ? "$gesamt Stelle(n) repariert.\n" : "$gesamt Stelle(n) gefunden. Mit --schreiben anwenden.\n";
+1 -1
View File
@@ -283,7 +283,7 @@ $theme->save(
['key' => 'body', 'name' => 'Fließtext', 'family' => 'Arial', 'source' => 'system', 'weights' => [400, 600, 700], 'fallback' => 'sans-serif'],
['key' => 'heading', 'name' => 'Überschriften', 'family' => 'Arial', 'source' => 'system', 'weights' => [400, 700], 'fallback' => 'sans-serif'],
],
['fontFamily' => 'body', 'headingFontFamily' => 'heading', 'fontSize' => '16px', 'lineHeight' => 1.6, 'color' => '@anthrazit', 'backgroundColor' => '@papier', 'linkColor' => '@anthrazit', 'containerWidth' => '1320px', 'headingScale' => 1.25, 'linkUnderline' => false],
['fontFamily' => 'body', 'headingFontFamily' => 'heading', 'fontSize' => '16px', 'lineHeight' => 1.6, 'color' => '@anthrazit', 'backgroundColor' => '@papier', 'linkColor' => '@anthrazit', 'containerWidth' => '1320px', 'headingScale' => 1.25, 'linkUnderline' => false, 'imageRadius' => '0px'],
['shape' => 'square', 'appearance' => 'solid', 'size' => 'normal', 'fontFamily' => 'inherit', 'fontSize' => 14, 'fontWeight' => 600, 'letterSpacing' => 0, 'textTransform' => 'none', 'backgroundColor' => '@blau-tief', 'color' => '@papier', 'hoverBackgroundColor' => '@blau-dunkel', 'hoverColor' => '@papier'],
);
echo "Farben, Schriften und Buttons gesetzt\n";
+152
View File
@@ -0,0 +1,152 @@
<?php
namespace App\Command;
use App\Content\DocumentValidator;
use Doctrine\DBAL\Connection;
use Symfony\Component\Console\Attribute\AsCommand;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;
/**
* Räumt Texte auf, die vor 1.20.1 bei jedem Speichern eine Kodierungsstufe dazubekommen haben.
*
* `RichText::sanitise()` hat Text ohne ein einziges Tag nur escaped, statt ihn vorher zu
* dekodieren. Aus „Freudenstadt &amp; Umgebung“ wurde deshalb `&amp;amp;`, dann `&amp;amp;amp;`
* – bis die Entity sichtbar auf der Seite stand. Der Fehler ist behoben; dieser Befehl bringt in
* Ordnung, was schon in der Datenbank liegt.
*
* Angefasst wird, was in Gebrauch ist: die veröffentlichte und die Entwurfsfassung jeder Seite
* und die Bereiche. Ältere Fassungen bleiben Historie – wer eine zurückholt, speichert sie
* danach einmal, und der reparierte Kern schreibt sie richtig.
*/
#[AsCommand(name: 'cms:repair:entities', description: 'Repariert doppelt kodierte Texte (&amp;amp;) aus Fassungen vor 1.20.1.')]
final class RepairEntitiesCommand extends Command
{
public function __construct(private Connection $db, private DocumentValidator $validator)
{
parent::__construct();
}
protected function configure(): void
{
$this->addOption('schreiben', null, InputOption::VALUE_NONE, 'Die Korrekturen wirklich speichern. Ohne diese Option wird nur gezeigt, was sich ändern würde.');
}
protected function execute(InputInterface $input, OutputInterface $output): int
{
$io = new SymfonyStyle($input, $output);
$schreiben = (bool) $input->getOption('schreiben');
$gesamt = 0;
$zeilen = [];
// Die aktiven Fassungen jeder Seite: veröffentlicht und Entwurf.
$aktive = [];
foreach ($this->db->fetchAllAssociative('SELECT id, path, published_revision_id, draft_revision_id FROM page') as $seite) {
foreach (['published_revision_id' => 'veröffentlicht', 'draft_revision_id' => 'Entwurf'] as $spalte => $was) {
if (!empty($seite[$spalte])) {
$aktive[(string) $seite[$spalte]] = '/'.$seite['path'].' ('.$was.')';
}
}
}
foreach ($aktive as $revision => $bezeichnung) {
$roh = $this->db->fetchOne('SELECT document FROM revision WHERE id = ?', [$revision]);
if (!is_string($roh)) {
continue;
}
$dokument = json_decode($roh, true, 512, JSON_THROW_ON_ERROR);
$zahl = 0;
$beispiele = [];
$dokument['blocks'] = $this->bausteine($dokument['blocks'] ?? [], $zahl, $beispiele);
if (0 === $zahl) {
continue;
}
$gesamt += $zahl;
$zeilen[] = [$bezeichnung, $zahl, implode(' · ', $beispiele)];
if ($schreiben) {
// Geprüft wie beim Speichern: eine Korrektur darf kein ungültiges Dokument hinterlassen.
$this->validator->validate($dokument, writing: true);
$this->db->executeStatement('UPDATE revision SET document = ? WHERE id = ?', [$this->json($dokument), $revision]);
}
}
foreach ($this->db->fetchAllAssociative('SELECT id, kind, name, document FROM partial') as $bereich) {
$dokument = json_decode((string) $bereich['document'], true, 512, JSON_THROW_ON_ERROR);
$zahl = 0;
$beispiele = [];
$dokument['blocks'] = $this->bausteine($dokument['blocks'] ?? [], $zahl, $beispiele);
if (0 === $zahl) {
continue;
}
$gesamt += $zahl;
$zeilen[] = ['Bereich '.$bereich['kind'].' „'.$bereich['name'].'“', $zahl, implode(' · ', $beispiele)];
if ($schreiben) {
// Ein Bereich traegt keine Ueberschrift - genau wie im PartialRepository.
$this->validator->validate($dokument, requireHeading: false, writing: true);
$this->db->executeStatement('UPDATE partial SET document = ? WHERE id = ?', [$this->json($dokument), $bereich['id']]);
}
}
if (0 === $gesamt) {
$io->success('Nichts zu reparieren: kein Text trägt eine doppelt kodierte Entity.');
return Command::SUCCESS;
}
$io->table(['Wo', 'Stellen', 'Beispiel'], $zeilen);
if ($schreiben) {
$io->success(sprintf('%d Stelle(n) repariert.', $gesamt));
return Command::SUCCESS;
}
$io->warning(sprintf('%d Stelle(n) gefunden. Mit --schreiben anwenden.', $gesamt));
return Command::SUCCESS;
}
/** @param array<int, array<string, mixed>> $liste */
private function bausteine(array $liste, int &$zahl, array &$beispiele): array
{
foreach ($liste as $index => $block) {
foreach (['text', 'caption', 'label', 'title', 'hint', 'provider'] as $feld) {
$wert = $block['props'][$feld] ?? null;
if (!is_string($wert) || '' === $wert) {
continue;
}
$neu = $this->entschachteln($wert);
if ($neu !== $wert) {
$liste[$index]['props'][$feld] = $neu;
++$zahl;
if (count($beispiele) < 3) {
$beispiele[] = mb_substr($wert, 0, 40).' → '.mb_substr($neu, 0, 40);
}
}
}
if (isset($block['children']) && is_array($block['children'])) {
$liste[$index]['children'] = $this->bausteine($block['children'], $zahl, $beispiele);
}
}
return $liste;
}
/** Entschachtelt so oft wie nötig: aus &amp;amp;amp; wird &amp;. Ein einzelnes &amp; ist richtig und bleibt. */
private function entschachteln(string $text): string
{
$vorher = null;
while ($vorher !== $text) {
$vorher = $text;
$text = preg_replace('/&amp;(amp|lt|gt|quot|apos|nbsp|#\d{1,6}|#x[0-9a-fA-F]{1,6});/', '&$1;', $text) ?? $text;
}
return $text;
}
/** @param array<string, mixed> $dokument */
private function json(array $dokument): string
{
return json_encode($dokument, JSON_THROW_ON_ERROR | JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES);
}
}
+6 -1
View File
@@ -68,6 +68,10 @@ final class Theme
'linkColor' => '@link',
'containerWidth' => '1240px',
'headingScale' => 1.25,
// Die Ecken jedes Bildes auf der Website. Stand bis 1.20.2 fest im Stylesheet -
// sichtbar gerundet, aber in keinem Feld zu finden. 12px bleibt die Vorgabe,
// damit bestehende Websites gleich aussehen; 0px macht Bilder eckig.
'imageRadius' => '12px',
// Unterstrichene Links im Fließtext. An, weil ein Link, den nur die Farbe verrät,
// für Menschen mit Farbsehschwäche kein Link ist. Wer eine ruhigere Seite will,
// schaltet es hier ab - dann bleibt der Unterstrich beim Überfahren.
@@ -229,7 +233,7 @@ final class Theme
}
$base[$field] = $value;
}
foreach (['fontSize', 'containerWidth'] as $field) {
foreach (['fontSize', 'containerWidth', 'imageRadius'] as $field) {
$value = trim((string) ($input[$field] ?? $base[$field]));
if (1 !== preg_match(StyleSchema::LENGTH, $value)) {
throw new InvalidPage(sprintf('„%s“ ist keine Größenangabe wie 18px oder 70rem.', $value));
@@ -338,6 +342,7 @@ final class Theme
}
$base = $theme['base'];
$variables[] = sprintf('--k-container:%s', $base['containerWidth']);
$variables[] = sprintf('--k-image-radius:%s', $base['imageRadius']);
$colour = static fn (string $v): string => str_starts_with($v, '@') ? sprintf('var(--k-%s)', substr($v, 1)) : $v;
+5
View File
@@ -162,6 +162,11 @@
<input id="base-scale" name="base[headingScale]" type="number" step="0.01" min="1" max="2" value="{{ theme.base.headingScale }}">
<p class="field-help">1,25 ergibt eine ruhige Staffelung, 1,5 eine sehr kontrastreiche.</p>
</div>
<div>
<label for="base-image-radius">Bildecken</label>
<input id="base-image-radius" name="base[imageRadius]" value="{{ theme.base.imageRadius }}" placeholder="12px">
<p class="field-help">Gilt für jedes Bild auf der Website – auch in Galerien, Logoleisten und Beitragslisten. <code>0px</code> macht sie eckig. Ein einzelnes Bild lässt sich im Editor unter <strong>Design → Bild → Bildecken</strong> abweichend einstellen.</p>
</div>
<div>
<label for="base-container">Inhaltsbreite</label>
<input id="base-container" name="base[containerWidth]" value="{{ theme.base.containerWidth }}" placeholder="1240px">
+28
View File
@@ -3111,6 +3111,34 @@ final class CmsTest extends TestCase
$code->assertScriptsUnchanged($added, $before);
}
public function testImageCornersComeFromTheDesignAreaAndCanBeSquare(): void
{
// Die Ecken jedes Bildes standen fest im Stylesheet: sichtbar gerundet, in keinem Feld
// zu finden, nur durch einen eigenen Wert am Block loszuwerden. Jetzt gehoeren sie dem
// Designbereich - mit derselben Vorgabe wie vorher, damit sich nichts veraendert.
$theme = $this->themes->load();
self::assertSame('12px', $theme['base']['imageRadius'], 'Die Vorgabe bleibt, was vorher im Stylesheet stand.');
$document = DemoContent::starter('Mit Bild');
$document['blocks'][0]['children'][] = ['id' => 'bild', 'type' => 'image', 'props' => ['src' => '/assets/image-placeholder.svg', 'alt' => 'Platzhalter', 'decorative' => false, 'width' => 1000, 'height' => 600]];
$id = $this->publishedPage('ecken', 'Ecken');
$stand = $this->pages->findForEditor($id);
$this->pages->publish($id, $this->pages->saveDraft($id, $document, $stand['version']), $this->pages->findForEditor($id)['version']);
self::assertStringContainsString('--k-image-radius:12px', $this->request('/ecken')->getContent());
// Eckig, wie es eine Vorlage ohne Rundungen braucht.
$this->themes->save($theme['colors'], $theme['fonts'], [...$theme['base'], 'imageRadius' => '0px']);
self::assertStringContainsString('--k-image-radius:0px', $this->request('/ecken')->getContent());
// Und was keine Groessenangabe ist, wird abgewiesen - wie bei der Inhaltsbreite.
try {
$this->themes->save($theme['colors'], $theme['fonts'], [...$theme['base'], 'imageRadius' => 'rund']);
self::fail('„rund“ ist keine Groessenangabe.');
} catch (InvalidPage) {
}
}
public function testSavingTextTwiceKeepsAmpersandsAndLineBreaksAsTheyAre(): void
{
// Umut: „Beim Startseitenbild wird & umgewandelt, aber nach dem Speichern steht &amp;
+107
View File
@@ -0,0 +1,107 @@
<?php
namespace App\Tests;
use App\Command\RepairEntitiesCommand;
use App\Content\DocumentValidator;
use App\Infrastructure\ConnectionFactory;
use Doctrine\DBAL\Connection;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Console\Tester\CommandTester;
/**
* `cms:repair:entities` räumt auf, was das doppelte Kodieren vor 1.20.1 hinterlassen hat.
*
* Geprüft wird beides: dass der Trockenlauf nichts anfasst, und dass das Schreiben genau die
* verschachtelten Entities entschachtelt – ein einzelnes `&amp;` ist richtig und muss bleiben.
*/
final class RepairEntitiesCommandTest extends TestCase
{
private string $path;
private Connection $db;
protected function setUp(): void
{
$this->path = tempnam(sys_get_temp_dir(), 'kant-repair-');
$this->db = ConnectionFactory::create('sqlite:///'.str_replace('\\', '/', $this->path));
TestSchema::build($this->db);
$this->db->insert('website', ['id' => 'default', 'name' => 'KantSites', 'default_locale' => 'de', 'theme' => '{}']);
}
protected function tearDown(): void
{
$this->db->close();
gc_collect_cycles();
@unlink($this->path);
}
/** Legt eine Seite mit einem Text an und gibt die Kennung der Fassung zurück. */
private function seite(string $text): string
{
$dokument = [
'schemaVersion' => 17,
'title' => 'Start',
'description' => 'Eine Seite zum Reparieren.',
'blocks' => [['id' => 's1', 'type' => 'section', 'children' => [
['id' => 'h1', 'type' => 'heading', 'props' => ['text' => 'Start', 'level' => 1]],
['id' => 't1', 'type' => 'text', 'props' => ['text' => $text]],
]]],
];
// Erst die Seite, dann ihre Fassung: die Fassung zeigt per Fremdschluessel auf die Seite.
$revision = bin2hex(random_bytes(16));
$this->db->insert('page', ['id' => 'page-1', 'slug' => 'home', 'path' => 'home', 'type' => 'page', 'locale' => 'de', 'website_id' => 'default', 'version' => 1, 'created_at' => gmdate('Y-m-d H:i:s')]);
$this->db->insert('revision', ['id' => $revision, 'page_id' => 'page-1', 'document' => json_encode($dokument, JSON_THROW_ON_ERROR), 'created_at' => gmdate('Y-m-d H:i:s')]);
$this->db->update('page', ['published_revision_id' => $revision, 'draft_revision_id' => $revision], ['id' => 'page-1']);
return $revision;
}
private function text(string $revision): string
{
$dokument = json_decode((string) $this->db->fetchOne('SELECT document FROM revision WHERE id = ?', [$revision]), true, 512, JSON_THROW_ON_ERROR);
return $dokument['blocks'][0]['children'][1]['props']['text'];
}
private function tester(): CommandTester
{
return new CommandTester(new RepairEntitiesCommand($this->db, new DocumentValidator()));
}
public function testTheDryRunReportsWithoutChangingAnything(): void
{
$revision = $this->seite('Freudenstadt &amp;amp; Umgebung');
$tester = $this->tester();
$tester->execute([]);
self::assertStringContainsString('1 Stelle', $tester->getDisplay());
self::assertSame('Freudenstadt &amp;amp; Umgebung', $this->text($revision), 'Ohne --schreiben bleibt alles stehen.');
}
public function testWritingUnnestsEveryLevelAndLeavesASingleEntityAlone(): void
{
// Drei Stufen, wie sie auf einer Seite standen, die dreimal gespeichert wurde.
$revision = $this->seite('Projekte &amp;amp;amp; Lösungen, „Zitat&amp;quot;, und ein richtiges &amp; dazu');
$tester = $this->tester();
$tester->execute(['--schreiben' => true]);
self::assertStringContainsString('repariert', $tester->getDisplay());
self::assertSame('Projekte &amp; Lösungen, „Zitat&quot;, und ein richtiges &amp; dazu', $this->text($revision));
// Ein zweiter Lauf findet nichts mehr - und macht aus dem richtigen &amp; kein nacktes &.
$zweiter = $this->tester();
$zweiter->execute(['--schreiben' => true]);
self::assertStringContainsString('Nichts zu reparieren', $zweiter->getDisplay());
self::assertSame('Projekte &amp; Lösungen, „Zitat&quot;, und ein richtiges &amp; dazu', $this->text($revision));
}
public function testAnUntouchedInstallationSaysSoAndKeepsQuiet(): void
{
$revision = $this->seite('Alles gut &amp; sauber');
$tester = $this->tester();
$tester->execute(['--schreiben' => true]);
self::assertStringContainsString('Nichts zu reparieren', $tester->getDisplay());
self::assertSame('Alles gut &amp; sauber', $this->text($revision));
}
}