THELIA Forum

Tout à fait, ce qui est parfaitement compréhensible!

Quand je dis que c'était plus simple sur Thelia 1, c'est qu'il suffisait d'ajouter un petit bout de code dans le plugin Colissimo (c'est mon seul transporteur) pour que ça marche. Là je t'avoue être complètement largué avec cette nouvelle version.

On avait un outil tout en français avec 1 forum et 1 zone pour trouver les modules. Maintenant on est sur un outil tout en anglais, avec une doc partiellement traduite en français, 1 forum, de nombreuses « pages » sur Github pour les modules qui sont souvent compatibles uniquement avec des versions qui ne sont même pas encore sorties sans que cela soit clairement indiqué, des mises à jour à faire en ligne de commande (pour le moment) pour une version 2.0 et non beta c'est assez étonnant je trouve. Quand j'ai découvert Thelia 1 j'étais impressionné par la simplicité d'utilisation, de développement et de possibilité qu'on nous offrait même sans avoir de connaissances techniques poussées. Thelia 2 est Open Source, mais pas Open tout court ^^ Ce qui est frustrant puisque cette nouvelle version offre des possibilités franchement excellentes! Bref, je m'éloigne du sujet initial...

Je viens de :
- désactiver le module
- supprimer le module
- faire un clear-cache.php
- supprimer tous les dossiers dans /cache (mais en fait y avais rien d'autres que le .htaccess que j'ai conservé)
- télécharger à nouveau le module depuis : https://github.com/thelia-modules/FreeShipping
- uploader le module (sans -master)
- activer le module
- passer en position 1
- ajouter la règle France et un montant de 10
- accès au panier via index_dev.php

À ce moment là, aucun message d'erreur, et aucun prix qui ne s'affiche. Je poursuis donc la commande, mais sur la page suivante on me propose le mode de livraison par Colissimo au tarif plein. Comme les frais de port sont normalement offerts à partir de 60€, je vais dans la configuration du module pour modifier le montant, j'ai un beau /admin/module/FreeShipping/save qui s'affiche en page blanche.

Last edited by iSkean (13-10-2014 13:30:26)

Bon iSkean va etre content !

je me suis penche sur le PB
j'en ai identifie une partie qui permet de le faire fonctionner

il faut modifier dans le module Freeshipping
fichier module->FreeShipping->FreeShipping.php
modifies la fonction qui suit
(elle renvoie si le module est valide pour la destination, par defaut elle renvoie vrai tout le temps, la suite corrige ce bug)

    public function isValidDelivery(Country $country)
    {
      $areaId = $country->getAreaId();
      $area = FreeShippingQuery::create()->findOneByAreaId($areaId);
      $maxAmount = $area->getAmount();
      
      $cart = $this->getContainer()->get('request')->getSession()->getCart();
      $amount = $cart->getTotalAmount();

      if(isset($area) && $amount>$maxAmount){
        return true;
      }
      return false;
    }

cette correction permet d'utiliser le module
mais il reste un autre bug
- il n'est pas possible de modifier la valeur du franco de port
la solution c'est de desactiver puis reactiver le module et reentrer la somme/zone

je m'y pencherai plus tard...qu'en j'aurais un moment
je ferai pullrequest a la fin

Petit rappel:
Thelia est livre sous licence LGPL
si on veut que cela continue et que le projet evolue
il ne faut pas que demander mais aussi participer a son developpement
surtout si on a des connaissances dans les langages web
ALORS CONTRIBUE !
voici le lien vers la doc

DOC

tu trouveras comment recuperer le total TTC (....une piste getTaxedAmount)
et ainsi modifier le fichier modules->FreeShipping->FreShipping.php

changes tous les getAmount()
par getTaxedAmount($country)

dans le fichier module->FreeShipping->FreeShipping.php

Last edited by zzuutt (15-10-2014 12:10:29)

Je découvre juste ce fil après avoir passé quelques jours à débugguer avec l'auteur (merci à lui), la version branche 2.0.4 marche maintenant nickel (et devrait pour tout le monde, mes templates sont simples). J'ai juste eu à faire une loop pour supprimer Colissimo quand on est en Free, et vice-versa (faire apparaitre le choix "franco" quand on est en dessous du seuil n'a pas de sens, tous les clients vont cliquer dessus ...). Voilà c'était juste pour info.

Voilà, copié-collé de tout ce qu'il y a dans la form dans ajax/order-delivery-module-list.hml . J'ai laissé ce qui est commenté pour voir ce qu'il y a d'enlevé par rapport template de base.

{form_field form=$form field='delivery-module'}
{*
        {if $isPost}
            {if $value == $ID}
                {assign var="isDeliveryMethodChecked" value="1"}
            {/if}
        {elseif $LOOP_COUNT == 1}
            {assign var="isDeliveryMethodChecked" value="1"}
        {/if}
*}
        
				{* si freeshipping affiche que ça sinon que colissimo  *}
				{loop type="module" name="codemodule" id="{$ID}"}
					{assign var="idDeliv" value="{$ID}"}
					{assign var="codeDeliv" value="{$CODE}"}
				{/loop}
				
				
				{loop name="franco" type="free_shipping"}
					{if {cart attr="total_taxed_price"} > {$AMOUNT} && {$codeDeliv} == "FreeShipping" }
						<label for="delivery-method_{$idDeliv}">
							<input type="radio" name="{$name}" id="delivery-method_{$idDeliv}" checked="checked" value="{$idDeliv}">
							<strong>{$TITLE}</strong>
						</label>
					{else if  {cart attr="total_taxed_price"} < {$AMOUNT} && {$codeDeliv} == "Colissimo" }
						<label for="delivery-method_{$idDeliv}">
							<input type="radio" name="{$name}" id="delivery-method_{$idDeliv}" checked="checked" value="{$idDeliv}">
							<strong>{$TITLE}</strong> / {format_money number=$POSTAGE symbol={currency attr="symbol"}}
						</label>
					{/if}
				{/loop}
					

					
      {/form_field}

J'ai aussi enlevé le "/" derrière "Franco de port", ou alors il faudrait faire "Franco /0" ..
Voilà, en espérant que ça servira !

Dans FreeShipping\FreeShipping.php,
$area = FreeShippingQuery::create()->findOneByAreaId($areaId);
$maxAmount = $area->getAmount();

Si le franco n'est pas disponible pour un pays, on a une erreur fatale sur la 2ème ligne car $area est null.

Ce test est toujours vrai. Il faut utiliser ça:  if (null !== $area) {