Visualizzazione di banner per parole chiave

[pstrada]

se ritieni che sia un problema fai una segnalazione al team di sviluppo

[cirosimeoli]

@pstrada ciao, grazie del consiglio. Mi spiegheresti come fare, perchè non saprei a chi rivolgermi. Grazie ancora

[pstrada]

Puoi inserire la segnalazione su github

https://github.com/joomla/joomla-cms/

[cirosimeoli]

@pstrada fatto. grazie. buona giornata

[cirosimeoli]

.. ed ecco il risultato:
https://github.com/joomla/joomla-cms/pull/45396. Ho provato ad emendare il file indicato (modules/mod_banners/src/Helper/BannersHelper.php) con il codice indicato, ma il risultato è stato che Joomla non trova più l'url degli articoli. forse ho capito male
.

[jabba]

Ciao, attiva il debug di Joomla (in configurazione globale) poi fai uno screenshot (o copia e incolla) dell'errore nella pull request, così dai qualche info a riguardo ai dev.
Ad ogni modo se c'è già una pull request probabilmente a breve verrà rilasciato in un prossimo aggiornamento. Ciao!

[cirosimeoli]

grazie Jabba. Il tuo commento mi rincuora. farò come mi suggerisci. Buona giornata

[Red]

Ciao, io ho fatto la modifica e non mi da errori, però non ho moduli banner attivi, questo il codice "corretto" completo

<?php

/**
 * @package     Joomla.Site
 * @subpackage  mod_banners
 *
 * @copyright   (C) 2006 Open Source Matters, Inc. <https://www.joomla.org>
 * @license     GNU General Public License version 2 or later; see LICENSE.txt
 */

namespace Joomla\Module\Banners\Site\Helper;

use Joomla\CMS\Application\CMSApplication;
use Joomla\CMS\Component\ComponentHelper;
use Joomla\CMS\Environment\Browser;
use Joomla\Component\Banners\Site\Model\BannersModel;
use Joomla\Registry\Registry;

// phpcs:disable PSR1.Files.SideEffects
\defined('_JEXEC') or die;
// phpcs:enable PSR1.Files.SideEffects

/**
 * Helper for mod_banners
 *
 * @since  1.5
 */
class BannersHelper
{
    /**
     * Retrieve list of banners
     *
     * @param   Registry        $params  The module parameters
     * @param   CMSApplication  $app     The application
     *
     * @return  mixed
     *
     * @since   5.1.0
     */
    public function getBanners(Registry $params, CMSApplication $app)
    {
        /** @var BannersModel $model */
        $model = $app->bootComponent('com_banners')->getMVCFactory()->createModel('Banners', 'Site', ['ignore_request' => true]);

                $input    = $app->input;
        $keywords = [];

        // Check if the current view is an article
        if ($input->getCmd('option') === 'com_content' && $input->getCmd('view') === 'article') {
            $articleId = $input->getInt('id');

            if (\is_integer($articleId)) {
                // Load the article model to get the keywords
                $contentModel = $app->bootComponent('com_content')->getMVCFactory()->createModel('Article', 'Site', ['ignore_request' => true]);

                $article  = $contentModel->getItem($articleId);
                $keywords = explode(',', $article->metakey);
            }
        }
        $config   = ComponentHelper::getParams('com_banners');

        $model->setState('filter.client_id', (int) $params->get('cid'));
        $model->setState('filter.category_id', $params->get('catid', []));
        $model->setState('list.limit', (int) $params->get('count', 1));
        $model->setState('list.start', 0);
        $model->setState('filter.ordering', $params->get('ordering'));
        $model->setState('filter.tag_search', $params->get('tag_search'));
        $model->setState('filter.keywords', $keywords);
        $model->setState('filter.language', $app->getLanguageFilter());

        $banners = $model->getItems();

        if ($banners) {
            if ($config->get('track_robots_impressions', 1) == 1 || !Browser::getInstance()->isRobot()) {
                $model->impress();
            }
        }

        return $banners;
    }

    /**
     * Retrieve list of banners
     *
     * @param   Registry        $params  The module parameters
     * @param   BannersModel    $model   The model
     * @param   CMSApplication  $app     The application
     *
     * @return  mixed
     *
     * @since   1.5
     *
     * @deprecated 5.1.0 will be removed in 7.0
     *             Use the non-static method getBanners
     *             Example: Factory::getApplication()->bootModule('mod_banners', 'site')
     *                          ->getHelper('BannersHelper')
     *                          ->getBanners($params, Factory::getApplication())
     */
    public static function getList(Registry $params, BannersModel $model, CMSApplication $app)
    {
        return (new self())->getBanners($params, $app);
    }
}

prova a fare un copia incolla totale, magari c'è qualche errore nel tuo file.

[cirosimeoli]

Buongiorno Red, grazie per essere intervenuto nella discussione. anche a me, pur avendo un modulo banner attivo, se non attivo l'opzione "ricerca per parola chiave", non dà errore; il problema si presenta appena attivo l'opzione! comunque, proverò a fare il copia/incolla del tuo codice e ti farò sapere. ancora grazie

[cirosimeoli]

niente da fare:

[jabba]

Su che versione di joomla stai facendo il test?

[cirosimeoli]

[Red]

potresti provare in ambiente di test un aggiornamento alla versione Nightly Builds 5.3.1 e vedere se è stato risolto definitivamente

[cirosimeoli]

Buongiorno Red. Grazie del post. A questo punto, preferisco aspettare.

[jabba]

Ciao @cirosimeoli , non so se stai ndo la PR su github ma qualcuno ha fatto una modifica ad un file 3 giorni fa, puoi provare a modificare con la nuova versione... ad ogni modo è una PR abbastanza seguita quindi sono convinto che salterà fuori in una versione ufficiale a breve. Ciao!

[cirosimeoli]

ciao Jabba. ok. grazie dell'aggiornamento. provo e vi faccio sapere

[cirosimeoli]

ciao Jabba. Ho provato col nuovo codice, ma niente da fare. Aspetteremo pazientemente le prossime releases. Grazie ancora