#623 - Konfiguration Teasereinstellungen

[sbn]

Hallo zusammen,

nach einem Upgrade von 4.10.1 (php 7.4.33) auf 4.10.2-dev (CONTENIDO-develop-before-php84-2025-02-02, php 8.2.30) ist mir aufgefallen, dass bei den Teasereinstellungen auf dem Tab "Einstellungen" des content_teaser_image Moduls zum Einen die Bildbreite und -skalierungseingabemöglichkeit fehlen, zum Anderen wird nur noch der ContentType für die "Quelle Text" angezeigt. "Quelle Überschrift", "Quelle Bild" und "Quelle Datum" fehlen.
Speichert man die Einstellungen erneut ab, gehen die nicht mehr sichtbaren Konfigurationen für Überschrift, Bild und Datum verloren.

Dies ist auch beim Beispielmandaten der Fall.

Hat das auch schon jemand bemerkt und gibt es dafür eine Lösung?

Viele Grüße
Sabine

4.10.2-dev.png (30.36 KiB) 355 mal betrachtet

4.10.1.png (41.7 KiB) 355 mal betrachtet

[Oldperl]

Hallo Sabine,

vielen Dank für Deine Fehlermeldung. Ich kann den Fehler bestätigen. Auch bei mir unter PHP 8.4 fehlen die entsprechenden Einträge.

Ich habe nun einen Eintrag im Bugtracker dazu angelegt https://github.com/CONTENIDO/CONTENIDO/issues/623 und das Thema ins entsprechende Forum verschoben.

Gruß aus Franken

Ortwin

[Faar]

Gibt es eine Fehlermedlung dazu im Logfile?

[Oldperl]

Hallo Faar,

nein, keine Meldung die man zuordnen könnte.

Gruß aus Franken

Ortwin

[sbn]

Hallo Faar,

ja auch bei mir leider keine Fehlermeldungen, die dem beschriebenen Fehler zuzuordnen wären.

Viele Grüße
Sabine

[Oldperl]

Servus,

Pull Request #627 erstellt, kommt nach Review in den develop-Branch https://github.com/CONTENIDO/CONTENIDO/pull/627

Gruß aus Franken

Ortwin

[sbn]

Ich habe das Problem beheben können. In der Funktion "private function _generateTabManual(): string" der Datei contenido/classes/content_types/class.content.type.teaser.php abZeil 1078 fehlten an einigen Stellen die Variablen "$wrapper" bzw. "$wrapperContent[]".

Nach Überarbeitung lautet die Funktion wie folgt:

private function _generateTabManual(): string
{
// define a wrapper which contains the whole content of the manual tab
$wrapper = new cHTMLDiv('', 'clearfix');
$wrapperContent = [];

$wrapperContent[] = new cHTMLParagraph(i18n("Content visualisation"), 'head_sub');

$wrapperContent[] = $this->makeFormRow([
new cHTMLLabel(i18n("Teaser visualisation"), $this->_getElementId('teaser_style')),
$this->_generateStyleSelect(),
]);

$wrapperContent[] = $this->makeFormRow([
new cHTMLLabel(i18n("Teaser filter"), $this->_getElementId('teaser_filter')),
new cHTMLTextbox(
'teaser_filter', $this->getSetting('teaser_filter'), '', '', $this->_getElementId('teaser_filter')
),
]);

$wrapperContent[] = $this->makeFormRow([
new cHTMLLabel(i18n('Character length'), $this->_getElementId('teaser_character_limit')),
new cHTMLTextbox(
'teaser_character_limit',
$this->getSetting('teaser_character_limit'),
'',
'',
$this->_getElementId('teaser_character_limit')
),
]);

$wrapperContent[] = new cHTMLParagraph(i18n("Pictures"), 'head_sub');
$wrapperContent[] = $this->makeFormRow([
new cHTMLLabel(i18n('Image width'), $this->_getElementId('teaser_image_width')),
new cHTMLTextbox(
'teaser_image_width',
$this->getSetting('teaser_image_width'),
'',
'',
$this->_getElementId('teaser_image_width')
),
]);
$wrapperContent[] = $this->makeFormRow([
new cHTMLLabel(i18n('Image height'), $this->_getElementId('teaser_image_height')),
new cHTMLTextbox(
'teaser_image_height',
$this->getSetting('teaser_image_height'),
'',
'',
$this->_getElementId('teaser_image_height')
),
]);


$wrapperContent[] = $this->makeFormRow([
new cHTMLLabel(i18n('Image scale'), $this->_getElementId('teaser_image_crop')),
$this->_generateCropSelect(),
]);

$wrapperContent[] = $this->makeFormRow([
new cHTMLLabel(i18n("Use original image"), $this->_getElementId('teaser_image_original')),
new cHTMLCheckbox(
'teaser_image_original',
'',
$this->_getElementId('teaser_image_original'),
($this->getSetting('teaser_image_original') == 'true')
),
]);

$wrapperContent[] = new cHTMLParagraph(i18n("Content types"), 'head_sub');
$this->makeFormRow([
$wrapperContent[] = new cHTMLLabel(i18n("Headline source"), $this->_getElementId('teaser_source_head')),
$wrapperContent[] = $this->_generateTypeSelect(
'teaser_source_head',
$this->getSetting('teaser_source_head'),
$this->getSetting('teaser_source_head_count')
),
]);

$this->makeFormRow([
$wrapperContent[] = new cHTMLLabel(i18n("Text source"), $this->_getElementId('teaser_source_text')),
$wrapperContent[] = $this->_generateTypeSelect(
'teaser_source_text',
$this->getSetting('teaser_source_text'),
$this->getSetting('teaser_source_text_count')
),
]);

$this->makeFormRow([
$wrapperContent[] = new cHTMLLabel(i18n('Image source'), $this->_getElementId('teaser_source_image')),
$wrapperContent[] = $this->_generateTypeSelect(
'teaser_source_image',
$this->getSetting('teaser_source_image'),
$this->getSetting('teaser_source_image_count')
),
]);

$this->makeFormRow([
$wrapperContent[] = new cHTMLLabel(i18n('Date source'), $this->_getElementId('teaser_source_date')),
$wrapperContent[] = $this->_generateTypeSelect(
'teaser_source_date',
$this->getSetting('teaser_source_date'),
$this->getSetting('teaser_source_date_count')
),
]);

$wrapper->setContent($wrapperContent);

return $wrapper->render();
}

[bodil]

Sehr gut! Vielen Dank! Ich hatte heute auch schon reingeschaut, bin so weit aber nicht gekommen.
Grüße!
bodil