Verschachteltes Accordion

pete_123 · 1. April 2020

Der Übersicht halber versuche ich es hier noch einmal (sorry für die obige Darstellung:)

Guten Abend zusammen,

ich habe aus dem Core Modul "Mod_Articles_News" des aktuellen Joomla ein Accordion-Modul erstellt (der Name lautet "mod_accordion", welches die Beiträge einer im Modul ausgewählten Kategorie anzeigt: Die Titel des Beitrages werden im Frontend als Button Überschrift angezeigt und der Inhalt des Beitrages im Slider des Accordions.

So weit so gut - es klappt alles (mit ein wenig Javascript aufgepeppt).

Nun möchte ich jedoch auch die Möglichkeit eines verschachtelten Accordions (innerem Accordion) hinzufügen.

Dazu habe ich in der "mod_accordion.xml" ein zusätzliches Feld eingefügt:

Code

<field

name="catid2"

type="category"

label="Kategorie 2"

description="Bitte tragen Sie hier die Unterkategorie ein, sofern Sie mit verschachtelten Akkordions arbeiten"

extension="com_content"

multiple="true"

filter="int_array"

class="multipleCategories"

/>

Alles anzeigen

Wie kann ich den Parameter des Feldes "catid2" ansprechen, dass dieser ebenfalls ausgegeben wird?

Der php Code in meiner "default.php" sieht wie folgt aus:

Spoiler anzeigen

Die "helper.php" hatte ich ebenfalls erweitert um die "public static function getSub":

Spoiler anzeigen

Im inneren Bereich des Accordion werden jetzt aber leider lediglich die Beitragstitel (als Button) der äußeren Kategorie ausgewiesen (also quasi doppelt).

Ich sehe momentan den Wald vor lauter Bäumen nicht mehr.

Was übersehe ich?

Vielen Dank für Eure Hilfe

Liebe Grüße Pete

LukasHH · 1. April 2020

Du hast ja die Zeile schon entsprechend markiert.

//Kategorie ID => Wechsel auf catid2 funktioniert nicht

Ich würde das Abrufen der Daten in eine eigene Funktion bauen und diese dann jeweils mit catid und ein zweites mal mit catid2 aufrufen.

pete_123 · 1. April 2020

Hallo Lukas,

danke für Deine schnelle Antwort.

In der "helper.php" aus der Joomla Core habe ich eine 2. Funktion eingebaut. Die native lautet public static function getList(&$params), wird mit "catid" angesprochen und meine hinzugefügte lautet public static function getSub(&$params), wird mit catid2 angesprochen.

Oder habe ich Dich falsch verstanden?

LukasHH · 1. April 2020

hast Du denn die zweite Funktion auch in der mod_articles_news.php aufgerufen?

Dort findest Du die Zeile

$list = ModArticlesNewsHelper::getList($params);

Genauso musst Du dann auch die zweite Funktion aufrufen:

PHP

$sub= ModArticlesNewsHelper::getSub($params);

Du könntest auch mit nur einer Funktion arbeiten, muss die Funktion getList nur um den Parameter $my_catid erweitern.

Beispiel:

PHP

public static function getList(&$params, $my_catid){
...
Zeile 56: $model->setState('filter.category_id', $my_catid);
...
}

Aufruf in der mod_articles_news.php dann:

PHP

$list            = ModArticlesNewsHelper::getList($params, $params->get('catid', array()));
$sub            = ModArticlesNewsHelper::getList($params, $params->get('catid2', array()));

pete_123 · 1. April 2020

Danke Lukas,

Zitat

hast Du denn die zweite Funktion auch in der mod_articles_news.php aufgerufen?

Dort findest Du die Zeile

$list = ModArticlesNewsHelper::getList($params);

Genauso musst Du dann auch die zweite Funktion aufrufen:

Ja, das hatte ich - leider ohne Erfolg.

Sympathischer ist mir aber Dein Ansatz, beide Parameter in einer Funktion aufzurufen.

Das werde ich jetzt ausprobieren und melde mich dann zurück.

Bis später

Meine helper.php sie nun wie folgt aus:

PHP

Meine mod_accordion.php:

PHP

Meine mod_accordion.xml:

XML

Und meine Default.php:

Spoiler anzeigen

Leider bekomme ich jetzt einen Fatal Error ausgewiesen?!

LukasHH · 1. April 2020

lasse mal die Inhalte der $list und $sub temporär ausgeben, um zu sehen, ob die variablen gefüllt sind.

In der default.php eintragen

PHP

<?php
echo '<pre style="white-space:pre !important;">';
print_r($list);
echo '</pre>';
echo '<pre style="white-space:pre !important;">';
print_r($sub);
echo '</pre>';
?>

Ich habe das bei mir nachgebaut und es funktioniert soweit.

LukasHH · 1. April 2020

In der default.php

würde ich auch aus:
<?php foreach ($sub as $item) : ?>

das machen:

<?php foreach ($sub as $subitem) : ?>

das $item hast Du schon in der übergeordneten Schleife.

pete_123 · 1. April 2020

Die Variablen sind beide gefüllt !!!

Zitat von LukasHH

In der default.php

würde ich auch aus:
<?php foreach ($sub as $item) : ?>

das machen:

<?php foreach ($sub as $subitem) : ?>

das $item hast Du schon in der übergeordneten Schleife.

Alles anzeigen

ja, danke für den Hinweise, habe ich nun auch abgeändert.

Zitat

Ich habe das bei mir nachgebaut und es funktioniert soweit.

wie sieht Deine helper.php denn genau aus?

LukasHH · 1. April 2020

Die Grundfunktion würde ich mir aber auch nochmal überlegen. Ich habe Kategorie eine ausgewählt und in Subkategorie ebenfalls.

Jetzt werden, so wie von Dir ausgegeben in jedem Beitrag die Beiträge der Subkategorie ausgegeben.

Hier mal bei mir die Testausgabe mit jeweils einer Kategorie. Ich habe das jetzt ohne der Accordion-Funktion und nur den Titel ausgeben lassen.

Das ist die helper.php

PHP: helper.php

<?php
/**
 * @package     Joomla.Site
 * @subpackage  mod_articles_news
 *
 * @copyright   Copyright (C) 2005 - 2019 Open Source Matters, Inc. All rights reserved.
 * @license     GNU General Public License version 2 or later; see LICENSE.txt
 */

defined('_JEXEC') or die;

JLoader::register('ContentHelperRoute', JPATH_SITE . '/components/com_content/helpers/route.php');

JModelLegacy::addIncludePath(JPATH_SITE . '/components/com_content/models', 'ContentModel');

/**
 * Helper for mod_articles_news
 *
 * @since  1.6
 */
abstract class ModArticlesNewsHelper
{
    /**
     * Get a list of the latest articles from the article model
     *
     * @param   \Joomla\Registry\Registry  &$params  object holding the models parameters
     *
     * @return  mixed
     *
     * @since 1.6
     */
    public static function getList(&$params, $my_catid)
    {
        // Get an instance of the generic articles model
        $model = JModelLegacy::getInstance('Articles', 'ContentModel', array('ignore_request' => true));

        // Set application parameters in model
        $app       = JFactory::getApplication();
        $appParams = $app->getParams();
        $model->setState('params', $appParams);

        $model->setState('list.start', 0);
        $model->setState('filter.published', 1);

        // Set the filters based on the module params
        $model->setState('list.limit', (int) $params->get('count', 5));

        // This module does not use tags data
        $model->setState('load_tags', false);

        // Access filter
        $access     = !JComponentHelper::getParams('com_content')->get('show_noauth');
        $authorised = JAccess::getAuthorisedViewLevels(JFactory::getUser()->get('id'));
        $model->setState('filter.access', $access);

        // Category filter
        //$model->setState('filter.category_id', $params->get('catid', array()));
        $model->setState('filter.category_id', $my_catid);

        // Filter by language
        $model->setState('filter.language', $app->getLanguageFilter());

        // Filer by tag
        $model->setState('filter.tag', $params->get('tag', array()));

        // Featured switch
        $featured = $params->get('show_featured', '');

        if ($featured === '')
        {
            $model->setState('filter.featured', 'show');
        }
        elseif ($featured)
        {
            $model->setState('filter.featured', 'only');
        }
        else
        {
            $model->setState('filter.featured', 'hide');
        }

        // Set ordering
        $ordering = $params->get('ordering', 'a.publish_up');
        $model->setState('list.ordering', $ordering);

        if (trim($ordering) === 'rand()')
        {
            $model->setState('list.ordering', JFactory::getDbo()->getQuery(true)->Rand());
        }
        else
        {
            $direction = $params->get('direction', 1) ? 'DESC' : 'ASC';
            $model->setState('list.direction', $direction);
            $model->setState('list.ordering', $ordering);
        }

        // Check if we should trigger additional plugin events
        $triggerEvents = $params->get('triggerevents', 1);

        // Retrieve Content
        $items = $model->getItems();

        foreach ($items as &$item)
        {
            $item->readmore = strlen(trim($item->fulltext));
            $item->slug     = $item->id . ':' . $item->alias;

            /** @deprecated Catslug is deprecated, use catid instead. 4.0 */
            $item->catslug  = $item->catid . ':' . $item->category_alias;

            if ($access || in_array($item->access, $authorised))
            {
                // We know that user has the privilege to view the article
                $item->link     = JRoute::_(ContentHelperRoute::getArticleRoute($item->slug, $item->catid, $item->language));
                $item->linkText = JText::_('MOD_ARTICLES_NEWS_READMORE');
            }
            else
            {
                $item->link = new JUri(JRoute::_('index.php?option=com_users&view=login', false));
                $item->link->setVar('return', base64_encode(ContentHelperRoute::getArticleRoute($item->slug, $item->catid, $item->language)));
                $item->linkText = JText::_('MOD_ARTICLES_NEWS_READMORE_REGISTER');
            }

            $item->introtext = JHtml::_('content.prepare', $item->introtext, '', 'mod_articles_news.content');

            // Remove any images belongs to the text
            if (!$params->get('image'))
            {
                $item->introtext = preg_replace('/<img[^>]*>/', '', $item->introtext);
            }

            // Show the Intro/Full image field of the article
            if ($params->get('img_intro_full') !== 'none')
            {
                $images = json_decode($item->images);
                $item->imageSrc = '';
                $item->imageAlt = '';
                $item->imageCaption = '';

                if ($params->get('img_intro_full') === 'intro' && !empty($images->image_intro))
                {
                    $item->imageSrc = htmlspecialchars($images->image_intro, ENT_COMPAT, 'UTF-8');
                    $item->imageAlt = htmlspecialchars($images->image_intro_alt, ENT_COMPAT, 'UTF-8');

                    if ($images->image_intro_caption) 
                    {
                        $item->imageCaption = htmlspecialchars($images->image_intro_caption, ENT_COMPAT, 'UTF-8');
                    }
                }
                elseif ($params->get('img_intro_full') === 'full' && !empty($images->image_fulltext))
                {
                    $item->imageSrc = htmlspecialchars($images->image_fulltext, ENT_COMPAT, 'UTF-8');
                    $item->imageAlt = htmlspecialchars($images->image_fulltext_alt, ENT_COMPAT, 'UTF-8');

                    if ($images->image_intro_caption) 
                    {
                        $item->imageCaption = htmlspecialchars($images->image_fulltext_caption, ENT_COMPAT, 'UTF-8');
                    }
                }
            }

            if ($triggerEvents)
            {
                $item->text = '';
                $app->triggerEvent('onContentPrepare', array ('com_content.article', &$item, &$params, 0));

                $results                 = $app->triggerEvent('onContentAfterTitle', array('com_content.article', &$item, &$params, 0));
                $item->afterDisplayTitle = trim(implode("\n", $results));

                $results                    = $app->triggerEvent('onContentBeforeDisplay', array('com_content.article', &$item, &$params, 0));
                $item->beforeDisplayContent = trim(implode("\n", $results));

                $results                   = $app->triggerEvent('onContentAfterDisplay', array('com_content.article', &$item, &$params, 0));
                $item->afterDisplayContent = trim(implode("\n", $results));
            }
            else
            {
                $item->afterDisplayTitle    = '';
                $item->beforeDisplayContent = '';
                $item->afterDisplayContent  = '';
            }
        }

        return $items;
    }
}

Alles anzeigen

pete_123 · 2. April 2020

Die helper.php ist identisch mit meiner....

In der default.php werden beide Variablen ($list und $sub) geprintet und sind mit den richtigen Kategorien gefüllt.

Die mod_accordion.php haben wir auch überprüft.

Wer findet den Fehler?!

Jetzt gibt es aber keine Fehlermeldung, dafür bleibt die Frontend Seite weiss.

Diese Zeilen in der helper.php verursachen einen Fehler:

PHP

// Category filter
        $model->setState('filter.category_id', $params->get('catid', array()));
        //$model->setState('filter.category_id', $my_catid);

Wenn ich die obere Zeile aktiviere erscheint folgendes Frontend:

Es werden die Beiträge der 1. Kategorie angezeigt, anschließend werden sie in den grauen Buttons wiederholt, statt dass die Subkategorie angezeigt wird.

Wenn ich vom obigen Code die erste Zeile deaktiviere und die zweite Zeile mit "$my_catid aktiviere, bleibt das Frontend leer.

LukasHH · 2. April 2020

Ich habe auf meinem Testsystem die mod_articles_news überarbeitet.

Das sind die beiden Felder in der XML (mod_articles_news.xml). Das erste ist das Original-Feld und das zweite habe ich hinzugefügt.

XML

<field
name="catid"
type="category"
label="JCATEGORY"
description="MOD_ARTICLES_NEWS_FIELD_CATEGORY_DESC"
extension="com_content"
multiple="true"
filter="int_array"
class="multipleCategories"
/>
<field
name="subcatid"
type="category"
label="SUBCATEGORY"
description="MOD_ARTICLES_NEWS_FIELD_CATEGORY_DESC"
extension="com_content"
multiple="true"
filter="int_array"
class="multipleCategories"
/>

Alles anzeigen

Hier die mod_articles_news.php

PHP

<?php
/**
* @package     Joomla.Site
* @subpackage  mod_articles_news
*
* @copyright   Copyright (C) 2005 - 2019 Open Source Matters, Inc. All rights reserved.
* @license     GNU General Public License version 2 or later; see LICENSE.txt
*/

defined('_JEXEC') or die;

// Include the news functions only once
JLoader::register('ModArticlesNewsHelper', __DIR__ . '/helper.php');

$list            = ModArticlesNewsHelper::getList($params, $params->get('catid', array()));
$sub            = ModArticlesNewsHelper::getList($params, $params->get('subcatid', array()));

$moduleclass_sfx = htmlspecialchars($params->get('moduleclass_sfx'), ENT_COMPAT, 'UTF-8');

require JModuleHelper::getLayoutPath('mod_articles_news', $params->get('layout', 'horizontal'));

Alles anzeigen

Dann habe ich die default.php kopiert und umbenannt nach accordeon.php, praktisch ein neues Layout erstellt. Das kann man ja im Modul auswählen. Zu Testzwecken, lasse ich dabei erst einmal nur den Titel ausgeben.

PHP

<?php
/**
 * @package     Joomla.Site
 * @subpackage  mod_articles_news
 *
 * @copyright   Copyright (C) 2005 - 2019 Open Source Matters, Inc. All rights reserved.
 * @license     GNU General Public License version 2 or later; see LICENSE.txt
 */

defined('_JEXEC') or die;

    echo '<pre style="white-space:pre !important;">';
    print_r($list);
    echo '</pre>';
        echo '<pre style="white-space:pre !important;">';
    print_r($sub);
    echo '</pre>';
    
?>

<div class="accordion-outer<?php echo $moduleclass_sfx; ?>">
    <?php foreach ($list as $item) : ?>
        <div class="accordion-item">
            <button>Kategorie: <?php echo $item->title; ?></button>
            <div class="panel content content-outer">
                <?php foreach ($sub as $subitem) : ?>
                <div style="margin-left: 50px;">
                    <button>Subkategorie: <?php echo $subitem->title; ?></button>
                </div>
                <?php endforeach; ?>
            </div>
        </div>
    <?php endforeach; ?>
</div>

Alles anzeigen

pete_123 · 2. April 2020

Danke Lukas für Deine umfangreiche Hilfestellung.

Bin leider erst jetzt wieder online.

Werde mir das gleich in Ruhe anschauen und nachstellen.

Melde mich dann wieder

Leider funktioniert es so nicht.

Folgendes habe ich ausprobiert:

Spoiler anzeigen

PHP: default.php

//echo '<pre style="white-space:pre !important;">';
//print_r($list);
//echo '</pre>';
//echo '<pre style="white-space:pre !important;">';
//print_r($sub);
//echo '</pre>';



//var_dump($sub);
//jexit();

?>




<div class="fraladesign-accordion fraladesign-accordion-outer<?php echo $moduleclass_sfx; ?>">
    <?php foreach ($list as $item) : ?>
            <div class="accordion-item">
                <button class="accordion-button accordion accordion-outer"><?php echo $item->title; ?></button>
                <div class="panel content content-outer">
                    <?php require JModuleHelper::getLayoutPath('mod_fraladesign_accordion', '_item'); ?>
                    
                        
                                            <?php foreach ($sub as $subitem) : ?>    
                                                                <div class="accordion-inner">
                                                                    <button class="accordion-inner"><?php echo $subitem->title; ?></button>
                                                                        <div class="panel content content-inner">
                                                                            <?php require JModuleHelper::getLayoutPath('mod_fraladesign_accordion', '_item'); ?>
                                                                        </div>
                                                                </div>
                                            <?php endforeach; ?>
                        
                </div>
                
                
                
            </div>
        
        
    <?php endforeach; ?>
    
    
    
    
</div>

Alles anzeigen

Ich habe nacheinander die Variable $list (Hauptkategorie) und $sub (Subkategorie) geprintet bei aktivem jexit.

Dadurch werden sie richtig ausgegeben => stimmt alles. Sobald ich jexit aukommentiere, wird nichts mehr angezeigt.

Meine mod_accordion.php hat weiterhin folgenden Code:

Spoiler anzeigen

Und meine Helper.php folgenden Code:

Spoiler anzeigen

PHP: helper.php

<?php
/**
 * @package     Joomla.Site
 * @subpackage  mod_fraladesign_accordion
 *
 * @copyright   Copyright (C) 2005 - 2019 Open Source Matters, Inc. All rights reserved.
 * @license     GNU General Public License version 2 or later; see LICENSE.txt
 */

defined('_JEXEC') or die;

JLoader::register('ContentHelperRoute', JPATH_SITE . '/components/com_content/helpers/route.php');


JModelLegacy::addIncludePath(JPATH_SITE . '/components/com_content/models', 'ContentModel');

/**
 * Helper for mod_articles_news
 *
 * @since  1.6
 */
abstract class ModArticlesNewsHelper
{
    /**
     * Get a list of the latest articles from the article model
     *
     * @param   \Joomla\Registry\Registry  &$params  object holding the models parameters
     *
     * @return  mixed
     *
     * @since 1.6
     */
    public static function getList(&$params, $my_catid)                                                                        //hinzugefügt: $my_catid
    {
        // Get an instance of the generic articles model
        $model = JModelLegacy::getInstance('Articles', 'ContentModel', array('ignore_request' => true));

        // Set application parameters in model
        $app       = JFactory::getApplication();
        $appParams = $app->getParams();
        $model->setState('params', $appParams);

        $model->setState('list.start', 0);
        $model->setState('filter.published', 1);

        // Set the filters based on the module params
        $model->setState('list.limit', (int) $params->get('count', 5));

        // This module does not use tags data
        $model->setState('load_tags', false);

        // Access filter
        $access     = !JComponentHelper::getParams('com_content')->get('show_noauth');
        $authorised = JAccess::getAuthorisedViewLevels(JFactory::getUser()->get('id'));
        $model->setState('filter.access', $access);

        // Category filter
        $model->setState('filter.category_id', $params->get('catid', array()));                                                
        //$model->setState('filter.category_id', $my_catid);                                                            // hinzugefügt komplette Zeile: bei Aktivierung => leeres Frontend, bei Deaktivierung => Frontend wird angezeigt, jedoch falsche Subkategorien (Hauptkategorie = Subkategorie)

        // Filter by language
        $model->setState('filter.language', $app->getLanguageFilter());

        // Filer by tag
        $model->setState('filter.tag', $params->get('tag', array()));

        // Featured switch
        $featured = $params->get('show_featured', '');

        if ($featured === '')
        {
            $model->setState('filter.featured', 'show');
        }
        elseif ($featured)
        {
            $model->setState('filter.featured', 'only');
        }
        else
        {
            $model->setState('filter.featured', 'hide');
        }

        // Set ordering
        $ordering = $params->get('ordering', 'a.publish_up');
        $model->setState('list.ordering', $ordering);

        if (trim($ordering) === 'rand()')
        {
            $model->setState('list.ordering', JFactory::getDbo()->getQuery(true)->Rand());
        }
        else
        {
            $direction = $params->get('direction', 1) ? 'DESC' : 'ASC';
            $model->setState('list.direction', $direction);
            $model->setState('list.ordering', $ordering);
        }

        // Check if we should trigger additional plugin events
        $triggerEvents = $params->get('triggerevents', 1);

        // Retrieve Content
        $items = $model->getItems();

        foreach ($items as &$item)
        {
            $item->readmore = strlen(trim($item->fulltext));
            $item->slug     = $item->id . ':' . $item->alias;

            /** @deprecated Catslug is deprecated, use catid instead. 4.0 */
            $item->catslug  = $item->catid . ':' . $item->category_alias;
            $item->catslug  = $item->subcatid . ':' . $item->category_alias;                                                //hinzugefügt: komplette Zeile

            if ($access || in_array($item->access, $authorised))
            {
                // We know that user has the privilege to view the article
                $item->link     = JRoute::_(ContentHelperRoute::getArticleRoute($item->slug, $item->catid, $item->subcatid, $item->language));        //hinzugefügt: $item->subcatid,
                $item->linkText = JText::_('MOD_ARTICLES_NEWS_READMORE');
            }
            else
            {
                $item->link = new JUri(JRoute::_('index.php?option=com_users&view=login', false));
                $item->link->setVar('return', base64_encode(ContentHelperRoute::getArticleRoute($item->slug, $item->catid, $item->subcatid, $item->language)));    //hinzugefügt: $item->subcatid,
                $item->linkText = JText::_('MOD_ARTICLES_NEWS_READMORE_REGISTER');
            }

            $item->introtext = JHtml::_('content.prepare', $item->introtext, '', 'mod_fraladesign_accordion.content');

            // Remove any images belongs to the text
            if (!$params->get('image'))
            {
                $item->introtext = preg_replace('/<img[^>]*>/', '', $item->introtext);
            }

            // Show the Intro/Full image field of the article
            if ($params->get('img_intro_full') !== 'none')
            {
                $images = json_decode($item->images);
                $item->imageSrc = '';
                $item->imageAlt = '';
                $item->imageCaption = '';

                if ($params->get('img_intro_full') === 'intro' && !empty($images->image_intro))
                {
                    $item->imageSrc = htmlspecialchars($images->image_intro, ENT_COMPAT, 'UTF-8');
                    $item->imageAlt = htmlspecialchars($images->image_intro_alt, ENT_COMPAT, 'UTF-8');

                    if ($images->image_intro_caption) 
                    {
                        $item->imageCaption = htmlspecialchars($images->image_intro_caption, ENT_COMPAT, 'UTF-8');
                    }
                }
                elseif ($params->get('img_intro_full') === 'full' && !empty($images->image_fulltext))
                {
                    $item->imageSrc = htmlspecialchars($images->image_fulltext, ENT_COMPAT, 'UTF-8');
                    $item->imageAlt = htmlspecialchars($images->image_fulltext_alt, ENT_COMPAT, 'UTF-8');

                    if ($images->image_intro_caption) 
                    {
                        $item->imageCaption = htmlspecialchars($images->image_fulltext_caption, ENT_COMPAT, 'UTF-8');
                    }
                }
            }

            if ($triggerEvents)
            {
                $item->text = '';
                $app->triggerEvent('onContentPrepare', array ('com_content.article', &$item, &$params, 0));

                $results                 = $app->triggerEvent('onContentAfterTitle', array('com_content.article', &$item, &$params, 0));
                $item->afterDisplayTitle = trim(implode("\n", $results));

                $results                    = $app->triggerEvent('onContentBeforeDisplay', array('com_content.article', &$item, &$params, 0));
                $item->beforeDisplayContent = trim(implode("\n", $results));

                $results                   = $app->triggerEvent('onContentAfterDisplay', array('com_content.article', &$item, &$params, 0));
                $item->afterDisplayContent = trim(implode("\n", $results));
            }
            else
            {
                $item->afterDisplayTitle    = '';
                $item->beforeDisplayContent = '';
                $item->afterDisplayContent  = '';
            }
        }

        return $items;
    }
}

Alles anzeigen

Frontend zeigt folgendes:

Die Hauptkategorie mit ihren 4 Beiträgen wird angezeigt. Die Subkategorie mit 1 Beitrag wird nicht angezeigt, statt dessen wird die Hauptkategorie wiederholt.

LukasHH · 2. April 2020

Ich sehe gerade, dass Du die Klasse in der helper.php scheinbar nicht richtig benannt hast. Dein Modul hat den Namen mod_accordion, dann muss auch alles zu diesem Namen benannt sein. Das gilt für die Helper-Klasse, LayoutPath usw.

In Deinem Modul mod_accordion.php rufst Du die Helper-Klasse des Modul mod_article_news.php auf
ModArticlesNewsHelper::getList()

Du musst Deine Klasse auf Deinem Modul anpassen/umbenennen und diese dann aufrufen.

abstract class ModAccordionHelper{

...

}

Und dann entsprechend aufrufen in der mod_accordion.php mit

$list = ModAccordionHelper::getList($params, $params->get('catid', array()));

$sub = ModAccordionHelper::getList($params, $params->get('subcatid', array()));

Indigo66 · 2. April 2020

OT: Ich habe hier mal einiges zusammengeführt und unnötiges gelöscht, damit es übersichtlicher wird.

pete_123 · 2. April 2020

Danke Tom !

@Lukas:

habe Deine Vorschläge aufgegriffen und die Klasse geändert und registriert => kein Erfolg.

Zudem habe ich mir das Frontend noch einmal mit den Firefox Entwickler-Werkzeugen angeschaut. Mein Ergebnis lautet wie folgt:

HK	SK	Ergebnis
1. Beitrag Hauptkategorie		korrekt
	1. Beitrag Subkategorie	falsch (1. Titel HK - Beitrag #1 HK )
	2. Beitrag Subkategorie	falsch (2. Titel HK - Beitrag #1 HK)
	3. Beitrag Subkategorie	falsch (3. Titel HK - Beitrag #1 HK)
	4. Beitrag Subkategorie	falsch (4. Titel HK - Beitrag #1 HK)
2. Beitrag Hauptkategorie		korrekt
3. Beitrag Hauptkategorie		korrekt
4. Beitrag Hauptkategorie		korrekt

LukasHH · 2. April 2020

Dann stimmt in der Ausgabe etwas nicht. Wenn die Variable $sub korrekt befüllt wird, aber trotzdem die Titel der Hauptkategorie ausgegeben werden, dann stimmt daran etwas nicht. Ich denke mal, dass Du statt der Subkategorie, dann die Hauptkategorie ausgeben lässt.

Hast Du Dir dahingehend auch einmal die "_item.php" angeschaut?
Die müsstest Du für die "Subkategorie" ebenfalls kopieren und entsprechend anpassen.

pete_123 · 2. April 2020

Habe mir die "_item.php" angeschaut, finde dort aber keinerlei Passagen, die ich ändern müsste.

Zitat

Dann stimmt in der Ausgabe etwas nicht. Wenn die Variable $sub korrekt befüllt wird, aber trotzdem die Titel der Hauptkategorie ausgegeben werden, dann stimmt daran etwas nicht. Ich denke mal, dass Du statt der Subkategorie, dann die Hauptkategorie ausgeben lässt.

Die Ausgabe wird ja über die "default.php" bestimmt, die ich weiter oben angezeigt habe.

Mir fällt nicht ein, wo ich diese noch weiter anpassen sollte ?!

LukasHH · 2. April 2020

In der default.php rufst Du die For Schleife auf:

PHP

<?php foreach ($sub as $subitem) : ?>

Innerhalb dieser lässt Du den Titel des jeweiligen "subitem" ausgeben

PHP

<?php echo $subitem->title; ?>

Danach möchtest Du den Beitrag des "subitem" ausgeben lassen. In dieser Zeile wird die Layout-Datei "_item.php" aufgerufen. Innerhalb des PHP Code dieser Datei wird aber nicht mehr das "subitem" angesprochen, sondern das "item" aus der übergeordneten (noch geöffneten) For Schleife.

PHP

<?php require JModuleHelper::getLayoutPath('mod_fraladesign_accordion', '_item'); ?>

Du müsstest diese Datei kopieren und z.B. umbenennen nach _subitem.php und die Zeile zum Aufruf ändern nach

PHP

<?php require JModuleHelper::getLayoutPath('mod_fraladesign_accordion', '_subitem'); ?>

Hier hast Du einige Einträge, die auf das aktuelle $subitem verweisen müssen und nicht auf das $item
z.B. Zeile 15

PHP

aus:
<?php if ($item->link !== '' && $params->get('link_titles')) : ?>
muss:
<?php if ($subitem->link !== '' && $params->get('link_titles')) : ?>

PHP

<?php if ($params->get('item_title')) : ?>
    <?php $item_heading = $params->get('item_heading', 'h4'); ?>
    <<?php echo $item_heading; ?> class="newsflash-title<?php echo $params->get('moduleclass_sfx'); ?>">
    <?php if ($subitem->link !== '' && $params->get('link_titles')) : ?>
        <a href="<?php echo $subitem->link; ?>">
            <?php echo $subitem->title; ?>
        </a>
    <?php else : ?>
        <?php echo $subitem->title; ?>
    <?php endif; ?>
    </<?php echo $subitem_heading; ?>>
<?php endif; ?>

<?php if ($params->get('img_intro_full') !== 'none' && !empty($subitem->imageSrc)) : ?>    
    <figure class="newsflash-image">
        <img src="<?php echo $subitem->imageSrc; ?>" alt="<?php echo $subitem->imageAlt; ?>">
        <?php if (!empty($subitem->imageCaption)) : ?>
            <figcaption>
                <?php echo $subitem->imageCaption; ?>
            </figcaption>
        <?php endif; ?>
    </figure>
<?php endif; ?>

<?php if (!$params->get('intro_only')) : ?>
    <?php echo $subitem->afterDisplayTitle; ?>
<?php endif; ?>

<?php echo $subitem->beforeDisplayContent; ?>

<?php if ($params->get('show_introtext', 1)) : ?>
    <?php echo $subitem->introtext; ?>
<?php endif; ?>

<?php echo $subitem->afterDisplayContent; ?>

<?php if (isset($subitem->link) && $subitem->readmore != 0 && $params->get('readmore')) : ?>
    <?php echo '<a class="readmore" href="' . $subitem->link . '">' . $subitem->linkText . '</a>'; ?>
<?php endif; ?>

Alles anzeigen

pete_123 · 2. April 2020

Lukas, vielen Dank für Deine unglaublichen Bemühungen.

Ich habe die von Dir angegebenen Codepassagen eingebaut.

Meine Dateien sehen jetzt wie folgt aus:

default.php

mod_fraladesign_accordion.xml

mod_fraladesign_accordion.php

Die Datei "_subitem.php" habe ich von Dir kopiert.

Nach wie vor ist im Frontend keine Veränderung zu erkennen.

In der "helper.php" habe ich alternierend Zeile 59 auskommentiert. Der Effekt ist, dass bei aktiver Zeile das Frontend weiss bleibt (ohne Fehlermeldung), ich dafür aber in der "default.php" die Variablen "$list" und "$sub" richtig gefüllt auslesen kann.

Ist diese Zeile 59 auskommentiert, ist das Accordion im Frontend zu sehen, aber die Subkategorie wird falsch ausgegeben.

Es ist zum Verzweifeln.

pete_123 · 2. April 2020

Konnte leider nicht alle Dateien hochladen, da die Menge auf 10.000 Zeichen begrenzt ist. Hier nun die beiden fehlenden Dateien:

helper.php

Code

abstract class ModFraladesignAccordionHelper
{
    /**
     * Get a list of the latest articles from the article model
     *
     * @param   \Joomla\Registry\Registry  &$params  object holding the models parameters
     *
     * @return  mixed
     *
     * @since 1.6
     */
    public static function getList(&$params, $my_catid)                                                                        //hinzugefügt: $my_catid
    {
        // Get an instance of the generic articles model
        $model = JModelLegacy::getInstance('Articles', 'ContentModel', array('ignore_request' => true));

        // Set application parameters in model
        $app       = JFactory::getApplication();
        $appParams = $app->getParams();
        $model->setState('params', $appParams);

        $model->setState('list.start', 0);
        $model->setState('filter.published', 1);

        // Set the filters based on the module params
        $model->setState('list.limit', (int) $params->get('count', 5));

        // This module does not use tags data
        $model->setState('load_tags', false);

        // Access filter
        $access     = !JComponentHelper::getParams('com_content')->get('show_noauth');
        $authorised = JAccess::getAuthorisedViewLevels(JFactory::getUser()->get('id'));
        $model->setState('filter.access', $access);

        // Category filter
        $model->setState('filter.category_id', $params->get('catid', array()));
        //$model->setState('filter.category_id', $my_catid);                                                                        // hinzugefügt komplette Zeile: wenn diese Zeile aktiv => Frontend weiss, aber Variable "list" und "sub" in der default.php richtig gefüllt. Wenn diese Zeile deaktiviert => Frontend startet mit dem Accordion aber den falschen Einträgen

        // Filter by language
        $model->setState('filter.language', $app->getLanguageFilter());

        // Filer by tag
        $model->setState('filter.tag', $params->get('tag', array()));

        // Featured switch
        $featured = $params->get('show_featured', '');

        if ($featured === '')
        {
            $model->setState('filter.featured', 'show');
        }
        elseif ($featured)
        {
            $model->setState('filter.featured', 'only');
        }
        else
        {
            $model->setState('filter.featured', 'hide');
        }

        // Set ordering
        $ordering = $params->get('ordering', 'a.publish_up');
        $model->setState('list.ordering', $ordering);

        if (trim($ordering) === 'rand()')
        {
            $model->setState('list.ordering', JFactory::getDbo()->getQuery(true)->Rand());
        }
        else
        {
            $direction = $params->get('direction', 1) ? 'DESC' : 'ASC';
            $model->setState('list.direction', $direction);
            $model->setState('list.ordering', $ordering);
        }

        // Check if we should trigger additional plugin events
        $triggerEvents = $params->get('triggerevents', 1);

        // Retrieve Content
        $items = $model->getItems();

        foreach ($items as &$item)
        {
            $item->readmore = strlen(trim($item->fulltext));
            $item->slug     = $item->id . ':' . $item->alias;

            /** @deprecated Catslug is deprecated, use catid instead. 4.0 */
            $item->catslug  = $item->catid . ':' . $item->category_alias;

            if ($access || in_array($item->access, $authorised))
            {
                // We know that user has the privilege to view the article
                $item->link     = JRoute::_(ContentHelperRoute::getArticleRoute($item->slug, $item->catid, $item->language));
                $item->linkText = JText::_('MOD_ARTICLES_NEWS_READMORE');
            }
            else
            {
                $item->link = new JUri(JRoute::_('index.php?option=com_users&view=login', false));
                $item->link->setVar('return', base64_encode(ContentHelperRoute::getArticleRoute($item->slug, $item->catid, $item->language)));
                $item->linkText = JText::_('MOD_ARTICLES_NEWS_READMORE_REGISTER');
            }

            $item->introtext = JHtml::_('content.prepare', $item->introtext, '', 'mod_fraladesign_accordion.content');

            // Remove any images belongs to the text
            if (!$params->get('image'))
            {
                $item->introtext = preg_replace('/<img[^>]*>/', '', $item->introtext);
            }

            // Show the Intro/Full image field of the article
            if ($params->get('img_intro_full') !== 'none')
            {
                $images = json_decode($item->images);
                $item->imageSrc = '';
                $item->imageAlt = '';
                $item->imageCaption = '';

                if ($params->get('img_intro_full') === 'intro' && !empty($images->image_intro))
                {
                    $item->imageSrc = htmlspecialchars($images->image_intro, ENT_COMPAT, 'UTF-8');
                    $item->imageAlt = htmlspecialchars($images->image_intro_alt, ENT_COMPAT, 'UTF-8');

                    if ($images->image_intro_caption) 
                    {
                        $item->imageCaption = htmlspecialchars($images->image_intro_caption, ENT_COMPAT, 'UTF-8');
                    }
                }
                elseif ($params->get('img_intro_full') === 'full' && !empty($images->image_fulltext))
                {
                    $item->imageSrc = htmlspecialchars($images->image_fulltext, ENT_COMPAT, 'UTF-8');
                    $item->imageAlt = htmlspecialchars($images->image_fulltext_alt, ENT_COMPAT, 'UTF-8');

                    if ($images->image_intro_caption) 
                    {
                        $item->imageCaption = htmlspecialchars($images->image_fulltext_caption, ENT_COMPAT, 'UTF-8');
                    }
                }
            }

            if ($triggerEvents)
            {
                $item->text = '';
                $app->triggerEvent('onContentPrepare', array ('com_content.article', &$item, &$params, 0));

                $results                 = $app->triggerEvent('onContentAfterTitle', array('com_content.article', &$item, &$params, 0));
                $item->afterDisplayTitle = trim(implode("\n", $results));

                $results                    = $app->triggerEvent('onContentBeforeDisplay', array('com_content.article', &$item, &$params, 0));
                $item->beforeDisplayContent = trim(implode("\n", $results));

                $results                   = $app->triggerEvent('onContentAfterDisplay', array('com_content.article', &$item, &$params, 0));
                $item->afterDisplayContent = trim(implode("\n", $results));
            }
            else
            {
                $item->afterDisplayTitle    = '';
                $item->beforeDisplayContent = '';
                $item->afterDisplayContent  = '';
            }
        }

        return $items;
        
        // ################################################### HINZUGEFÜGT ######################################################################
        
        
    }
}

Alles anzeigen

Die Datei "_subitem.php" habe ich von Dir kopiert.

Tags