Module marqué comme "Sans surcharge" (Override) sur la Marketplace, alors qu'il contient plein de surcharge

[Axelweb]

Bonjour,

Sur la PrestaShop Marketplace, on peut trouver des modules (payants), étant indiqués comme "Sans surcharge", comme sur la capture d'écran, avec le module "Pretty URL".

Mais le module ajoute clairement des fichiers de surcharge dans le dossier /override, qui s'affiche dans Paramètres avancés -> Informations, bloc "Liste des surcharges", comme on peut le voir sur la seconde capture d'écran.

Encore mieux, quand on regarde un peu le code (voir ci-dessous), on s'aperçoit que pendant l'installation, le module installe toute une liste de fichier dans le dossier /override, et cela si la version est supérieur à 1.7.0.0, donc ce n'est pas une question de rétrocompatibilité, car sur un PrestaShop 8.2, les Overrides seront installés quand même.

Ma question est la suivante : Savez-vous pourquoi le module est quand même marqué comme "Sans surcharge" ? Est-ce le développeur qui indique cela, ou une vérification automatique de PrestaShop ?

Extrait du module "Pretty URL" :

	public function moveFiles()
	{
		if (Tools::version_compare(_PS_VERSION_, '1.7.0.0', '>=')) {
			Tools::deleteFile(_PS_OVERRIDE_DIR_.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'ProductController.php');
			Tools::deleteFile(_PS_OVERRIDE_DIR_.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'CategoryController.php');
			Tools::copy(_PS_MODULE_DIR_.$this->name.DIRECTORY_SEPARATOR.'override'.DIRECTORY_SEPARATOR.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'CategoryController.php',
						_PS_OVERRIDE_DIR_.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'CategoryController.php');
			Tools::copy(_PS_MODULE_DIR_.$this->name.DIRECTORY_SEPARATOR.'override'.DIRECTORY_SEPARATOR.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'ManufacturerController.php',
						_PS_OVERRIDE_DIR_.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'ManufacturerController.php');
			Tools::copy(_PS_MODULE_DIR_.$this->name.DIRECTORY_SEPARATOR.'override'.DIRECTORY_SEPARATOR.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'SupplierController.php',
						_PS_OVERRIDE_DIR_.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'SupplierController.php');
			Tools::copy(_PS_MODULE_DIR_.$this->name.DIRECTORY_SEPARATOR.'includes'.DIRECTORY_SEPARATOR.'ProductController.php',
						_PS_OVERRIDE_DIR_.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'ProductController.php');
			if (Tools::version_compare(_PS_VERSION_, '1.7.4.0', '<')) {
				Tools::copy(_PS_MODULE_DIR_.$this->name.DIRECTORY_SEPARATOR.'includes'.DIRECTORY_SEPARATOR.'CartController.php',
						_PS_OVERRIDE_DIR_.'controllers'.DIRECTORY_SEPARATOR.'front'.DIRECTORY_SEPARATOR.'CartController.php');
			}
		}
		return true;
	}

Le module étant indiqué comme "Sans surcharge", et les surcharges que celui-ci ajoute, que l'on peut voir depuis le back-office (Paramètres avancés -> Informations)

[Eolia]

Plusieurs remarques:

- Ce module est inutile et ne changera RIEN au référencement, c'est une aberration de croire que ça gène Google qu'il y ait un chiffre dans l'url, on a eu depuis 2016 de nombreux débats à ce sujet.

- Si vous avez des catégories avec nom identiques vous allez avoir de mauvaises surprises (Homme/teeshirt, Femme/teeshirt par exemple)

- Certains modules ne seront plus fonctionnels à cause de la ré-écriture des routes.

- Ce module installe ses overrides comme un sagouin en les écrivant de force plutôt que d'utiliser le fonctionnement natif de Prestashop (qui contrôle déjà que les fonctions ne sont pas déjà overridées)

[Axelweb]

10 minutes ago, Eolia said:

Plusieurs remarques:

- Ce module est inutile et ne changera RIEN au référencement, c'est une aberration de croire que ça gène Google qu'il y ait un chiffre dans l'url, on a eu depuis 2016 de nombreux débats à ce sujet.

- Si vous avez des catégories avec nom identiques vous allez avoir de mauvaises surprises (Homme/teeshirt, Femme/teeshirt par exemple)

- Certains modules ne seront plus fonctionnels à cause de la ré-écriture des routes.

- Ce module installe ses overrides comme un sagouin en les écrivant de force plutôt que d'utiliser le fonctionnement natif de Prestashop (qui contrôle déjà que les fonctions ne sont pas déjà overridées)

Hello Eolia, merci beaucoup pour ta réponse, c'est très intéressant. En effet, c'est le module qui ajoute le plus d'override actuellement, avec JPresta Page Cache Ultimate ...

[Prestashop Addict]

Pour ce genre de problème contactez directement Prestashop addons et signalez le problème. En fait sur addons la première publication d'un module passe dans un contrôle qualité "sérieux". MAIS à chaque mise à jour du module sur la market place, le contrôle est très sommaire, d'où votre mauvaise surprise 😞 Et c'est la responsabilité du développeur de mettre à jour les informations de son module.

[Eolia]

il y a 36 minutes, Axelweb a dit :

avec JPresta Page Cache Ultimate ...

Deuxième module inutile... Si votre site rame il faut en trouver la raison (Module mal écrit, requêtes SQL pas optimisées, serveur limité, etc...)

Ce genre de module ne sert que de cache misère mais ne corrige rien.

[Prestashop Addict]

il y a 3 minutes, Eolia a dit :

Ce genre de module ne sert que de cache misère mais ne corrige rien.

Je mesurerai un peu les propos de @Eolia, car pour des sites avec peu de ressources serveur (et il y énormément de petits sites sur de l'hébergement mutualisé), cela améliore grandement la vitesse du site. Pour ma part je l'ai testé sur plusieurs sites et franchement ça décoiffe. la majorité des pages sont en statique ! Et les overrides sont hélas nécessaire car Prestashop n'a pas prévu encore tous les hooks possibles pour ce type d'outils qui optimise en profondeur.

Quand on voit le nombre de requêtes SQL en mode profiling sur une simple page d'accueil qui change très rarement, il faut un serveur dédié et un serveur costaud et SGBD dédié pour avoir des performances acceptables, et c'est un budget pour des petites structures. Prestashop ne devrait pas selon moi, être l'apanage que des grands commerçants qui ont les moyens, cela va à l'encontre de la philosophie open source.

[Eolia]

Je ne suis pas d'accord. J'ai plusieurs clients chez O2switch (7€ par mois) et aucun problèmes de perfs. (Je ne parle pas d'OVH)

Et pour être intervenu de nombreuses fois, c'était à chaque fois un problème de script (dernier module débugué, avant intervention 40 secondes après debug 286 ms)

Pas d'index sur les tables, requêtes toutes en LEFT etc. (ça fonctionne avec John Doe et 3 robes mais avec 30 000 commandes, 65 000 produits ce n'est plus du tout la même histoire).

Si un Prestashop ne rame pas à l'install il ne doit pas ramer ensuite ou alors il faut regarder ce qu'on a modifié/ajouté.

[Axelweb]

21 hours ago, Prestashop Addict said:

Je mesurerai un peu les propos de @Eolia, car pour des sites avec peu de ressources serveur (et il y énormément de petits sites sur de l'hébergement mutualisé), cela améliore grandement la vitesse du site. Pour ma part je l'ai testé sur plusieurs sites et franchement ça décoiffe. la majorité des pages sont en statique ! Et les overrides sont hélas nécessaire car Prestashop n'a pas prévu encore tous les hooks possibles pour ce type d'outils qui optimise en profondeur.

Quand on voit le nombre de requêtes SQL en mode profiling sur une simple page d'accueil qui change très rarement, il faut un serveur dédié et un serveur costaud et SGBD dédié pour avoir des performances acceptables, et c'est un budget pour des petites structures. Prestashop ne devrait pas selon moi, être l'apanage que des grands commerçants qui ont les moyens, cela va à l'encontre de la philosophie open source.

En effet, le gain avec un cache statique peut être vraiment impressionnant. Mais vu tous les overrides qu'ajoute le module Page Cache Ultimate, je préfère m'en passer et utiliser Varnish par exemple, mais demande un peu plus de configuration (il ne faut pas mettre en cache les paniers, notifications ...).

Si vous pouvez mettre le lien de contact PrestaShop addons, je suis preneur, il est HS sur votre post

[Axelweb]

21 hours ago, Eolia said:

Deuxième module inutile... Si votre site rame il faut en trouver la raison (Module mal écrit, requêtes SQL pas optimisées, serveur limité, etc...)

Ce genre de module ne sert que de cache misère mais ne corrige rien.

Je suis d'accord, je n'ai pas choisi d'installer ce module, il était déjà présent quand j'ai commencé à travailler sur ce PrestaShop, mais on souhaite s'en débarrasser lors de la montée en version 8.2

[Divine]

Varnish n'est pas beaucoup mieux. Personnellement, je suis d'accord avec @Eolia, ce sont des caches misère. Une boutique correctement installée et configurée, avec des modules et des thèmes correctement développés et le tout tournant sur un bon hébergement ne devrait pas avoir besoin de ces caches.

[coeos.pro]

HEUREUSEMENT on paye 99 € pour une validation de QUALITE !!!