Jump to content

Julien Bourdeau

Members
  • Posts

    67
  • Joined

  • Last visited

  • Days Won

    1

Everything posted by Julien Bourdeau

  1. What do you mean? I don't get it. I'll have a look for the rest.
  2. Hi, When you install a theme, the custom modules defined by the theme as dependencies are installed. In 1.7, the theme maker can define what he wants to achieve with modules: enable/disable, hook/unhook, and so on. If there is any special modules, they are bundled with the theme and installed at the same time. Everything happens in the theme.yml: http://developers.prestashop.com/themes/gettingstarted/index.html#theme-yml We thought it didn't make much sense to ask anything to the use at this step. If you want to disable a module afterwards, you can do it in your module page of course. I hope this answers your question.
  3. Ah oui ok. C'est pas ce que j'avais compris. Donc non c'est pas possible, je ne peux pas m'engager mais à mon avis ça ne le sera jamais dans le core avec le theme par défaut (trop spécifique). Je pense que par contre prestashop offre la possiblité à ton theme d'implementer ce genre de comportement.
  4. Déjà merci pour ton message, mais ta compaison n'est pas la bonne. La bonne analogie c'est: "c'est un peu comme si vous disiez à un type qui fait une cuisine honnête qu'il ne peut pas être bénévole dans votre resto sous prétexte qu'il ne fait pas de l'assez bonne qualité". C'est ça la situation, et du coup ca me parait quand meme moins choquant. Le fait que ca soit plus rapide pour nous que pour toi ne me semble absolument pas un arguement valide. En soit j'aimerais bien passer plus de temps à debugger et merger avec la communauté (comme tout le monde ici) mais malheureseuement le temps manque cruellement. Par contre bonne nouvelle, Github nous permet depuis peu d'editer les PR des contributeurs, ça devrait regler ce problème.
  5. C'est possible puisque toutes les declinaisons ont une url unique (basé sur l'id combinaison qui a ete rajouté dans l'url). J'ai bien compris la question? Sinon je veux bien un peu plus d'explications.
  6. Hi, Here is the pull request, it will be merged soon. Once merged, you can export your theme with this command: "php app/console prestashop:theme:export themename" :tada:
  7. Have you actually looked at it? I really don't get how you can say that it's not easier/faster. Look at ANY template in default-bootstrap and compare it to the 1.7 template (either StarterTheme or Classic, I guarantee you'll find it **much** cleaner and clearer. I understand it can be hard to re-learn and it's not always nice to see things changing, but if you make this little effort at the beginning, you'll see that you'll never touch default-bootstrap ever again.
  8. Maintaining 2 modules might be complicated but maintaining backward compat' is WAY worse.
  9. 1. you should create a thread dedicated for this I would be happy to share my preferences and I guess many people can join in. 2. Not yet, but I'd like to make one. 3. The best way so far is to create a custom module to do that. Every developer will create their own. We would like to have a special kind of module, a "theme option module" 4. Nope. StarterTheme is agnostic, there is no lib bundled at all. The best way will be to use npm to get bootstrap 4, then import it in your theme.css and theme.js. Then modify templates to use bootstrap classes. 1. Thomas is working hard on the module validator right now, it should be in production soon. 2. Having a module compatible with both version is a **very very very very bad idea**. 1.6 will be maintained for a long time but there is no new features and such so your module shouldn't require much modification. Since 1.7 is a major version, loads of things have changed, like PHP version for example. There is also loads of great new features for front office: widgets, smarty is automatically escaped in 1.7, many smarty functions has been removed. If you want to have your module compatible, it will be hard, plus you won't be able to use ANY new feature (for example because namespace will break on PHP < 5.4. If you want to I believe the best way will be to have 2 modules.
  10. Effectivement il y a 24h de cache. Si tu connais une solution. je suis preneur.
  11. I'm saying it won't during the whole 1.7 cycle. Since we broke compatibility with all existing themes, I believe that it won't be for PrestaShop 1.8 either. Breaking compatibility for 1.7 was required to clean everything but breaking compatibility again to move from smarty to twig seems unlikely (it doesn't bring anthing to the table).
  12. It's true that 1.7 isn't about bringing bugfix on every subject but it doesn't mean we doesn't fix anything! There are plenty of bugs related to the former product.js for example, they have all gone away! 1.7 fixes a hell lot of bugs (at least on the front office part). But yeah, some topics haven't got much attention, including ASM. About Smarty vs Twig, I haven't compared much (I just checked as we're talking about it). Smarty seems a bit faster, okay, but twig documentation is much better and it's used by a lot of frameworks. I think PrestaShop should use more standard and common technologies. Twig performances are still very good. Twig 2 is under development, maybe it will be faster. Also, front office is still using Smarty and I think it's were performance matter the most.
  13. Please don't forget the StarterTheme and almost full rewrite of the front office part. The point is to improve stability and fix LOADS of bugs. it will also make template makers' life easier and improve module compatibility. It will probably introduce some flaws but I believe most of them will be fixed by core update (controller, presenters, objectmodels) without any theme update. I am deeply convinced this is a very very important step for PrestaShop ecosystem: hence the module and theme validation.
  14. Je veux bien des exemples précis stp. On a porté tous les modules les plus utiles. On ne peut pas tout faire dans une version! Si on a viré des modules comme themeconfigurator ou blockstores c'est justement parce qu'on voudrait refaire un système beaucoup plus simple et puissant. Par exemple blockstores, editorial et tout un tas d'autres modules similaires peuvent etre remplacer un module de contenu. Il faut aussi cette fameuse feature de hooker un module plusieurs fois sur le meme hook. L'intéret de cette 1.7 c'est poser des bases pour le futur. Refaire live edit ou la gestion du contenu sera beaucoup plus simple avec ces bases saines.
  15. Il faut bien comprendre les differents scénarii. Tu hook ton module sur un hook connu (genre displayLeftColumn) et tu geres le cas dans renderWidget() donc l'utilisateur final peut utiliser ton module sur n'importe quel hook. Tu peux même faire du preg_match sur le hookname, par exemple le comportement different sur un hook qui contient "column" ou "footer". Il faut quand meme toujours un comportement par defaut. Pour gerer la compat' avec un module 1.6, le mieux c'est de definir les methodes hookDisplayxxx qui fait un simple appel à renderWidget avec les bons parametres NOTE: Du coup tu ne peux pas implémenter l'interface puisqu'elle n'existe pas en 1.6. Il faut faire les methodes mais sans le "implement". Dans ton theme tu peux definir des hooks custom pour rajouter des features. En 1.7 c'est cool grace aux widget, en 1.6 il te faudra des modules compatibles (donc que les tiens avec une methode hookDisplayCustomxxx() ) Ensuite pour le {widget name="socialsharing"}: l'intéret c'est de pouvoir placer un module n'importe ou dans un theme. Ca ne crée pas de hook, ça appelle juste le module. C'est une feature qui est destiné aux marchants qui font un peu de custom eux meme ou des agences qui font un theme pour un client. Il vaut mieux l'éviter pour un theme Addons puisque ca n'est pas personnalisable dans la page module position, il faut forcement passer par le code.
  16. L'interet d'utiliser render widget c'est pouvoir hooker ton module sur un autre hook, y compris un hook custom d'un theme: hookDisplayProductDescriptionAfter par exemple. Si tu gardes la fonction hookxxx ton module ne marchera QUE sur le hook qui porte tres exactement ce nom. Du coup je vois pas trop de bonne raison de ne pas utiliser renderWidget ^^
  17. Effectivement tout l'intérêt de pouvoir casser la compat' était de pouvoir réorganiser tout ce qui est passé à Smarty et d'éviter les globales dans tous les sens. Toutes les URLS sont maintenant dans `$urls` et si besoin il y a un helper smarty {url}. Pour les fonctions sur les prix, elles ont été retiré du front (toujours dispo pour les PDF et l'Admin). Normalement les prix sont sensés arrivés dans la bonne currency et formatée. Si tu veux ajouter des trucs dans ton module et l'assigner dans smarty, tu peux le formaté coté PHP. Je t'encourage même à créer un presenter. Normalement ca marche exactement de la même manière.
  18. Ces exemples sont des choix UX pour notre theme par default mais tout reste faisable dans votre theme. Par contre on a effectivement supprimé quelques features, comme la carte google maps sur la pages stores. (note: on a remis catalog et b2b que très récemment)
  19. Next week i'm getting on documentation, I have loads of stuff I want to write. Also I thought about making a command for it like `php app/console prestashop:theme:release mythemename`, I hope I can do it
  20. Pour le moment l'objectif est "Automne 2016", j'ai rien de plus précis. C'est pour démarrer un projet?
  21. L'archive est téléchargé chez unicode.org, parfois le site ne repond pas, ou le telechargement s'arrete. Pour l'instant ça n'arrive pas trop souvent, j'espere que ca ne va pas empirer. Tu peux supprimer le fihcier corrompu et relancer composer install, et sinon meme le telecharger à la main dans le dossier avant de relancer composer: http://www.unicode.org/Public/cldr/26/json-full.zip
  22. Il vaut mieux tester sur une git. Tu devrais passer un peu de temps à mettre en place ton environnement, comme ca tu pourras récupérer les fixes facilement. En ce moment on corrige beaucoup de bug, donc quelques jours après la sortie de la beta elle est "périmé" puisque plein de chose ont déjà été corrigé. Si tu prends la dernière version du StarterTheme, mais pas la dernière version git du coeur, tu vas avoir des problèmes aussi. Concernant les fonctionnalités. La feature theme et grid va être HYPEP facile à reimplementer, si tu regardes les templates, tu verras que toutes les pages de liste heritent de "catalog/product-list.tpl", et que chaque thumbnail est crée à partir de "catalog/_partials/miniatures/product.tpl". En créant un template pour les miniatures en mode liste, et avec quelques lignes de JS tu peux proposer cette feature dans tes themes. https://github.com/PrestaShop/PrestaShop/blob/develop/themes/classic/templates/catalog/_partials/miniatures/product.tpl Pour l'ajax dans le panier normalement c'est bon, ca marche bien avec les + et - par contre je sais plus si on a corrigé le bug avec la saisie d'une valeur dans le champs. Faut que je regarde. Pour les images des déclinaisons, ça a été amélioré récemment: https://github.com/PrestaShop/PrestaShop/pull/5959 https://github.com/PrestaShop/PrestaShop/pull/5971
  23. Oui c'est possible. Le mieux c'est vriament de maintenir 2 templates differents, 1 pour la 1.6 avec des |escape:HTML:UTF8 notamment et un template plus clean pour la 1.7. Il suffit d'avoir 2 dossiers /views/templates/1.6/front/xx et /views/templates/1.7/front/xxx. Pour le system de traduction, on a maintenu la compatibilité le $this->l() donc à mon avis, le mieux c'est de ne pas utiliser le nouveau systeme de traduction si tu veux module compatible 1.6 et 1.7. Ensuite tu peux quand meme utiliser le system de widget en créant les 2 methodes requises pour faire un widget (getWidgetVariables et renderWidget) et ensuite appeler ces methods dans les methods hookDisplayXxx. Par contre, il ne faut pas implementer l'interface PHP puisqu'elle n'existe pas en 1.6 https://github.com/PrestaShop/docs/blob/master/module/widget.rst J'ai ajouté une page todo dans la doc basé sur tes remarques. https://github.com/PrestaShop/docs/blob/master/module/module-backward-compat.rst
  24. Pour le starter theme, en ce moment ca peut arriver qu'il ait quelques jours de retard sur le core. Prend bien une dernière branche develop de PrestaShop/PrestaShop et la dernière master du StarterTheme. Notamment pour ces 2 PR qui ont été mergées hier et aujourd'hui: https://github.com/PrestaShop/StarterTheme/pull/100 https://github.com/PrestaShop/StarterTheme/pull/101 Pour info on teste le StarterTheme sur les PR PrestaShop maintenant donc ca ne devrait plus ce produire (https://github.com/PrestaShop/PrestaShop/pull/6038) Les crashs en cause pour la page categorie, ça vaut quand même le coup, je pense que ça apporte une nouvelle feature vraiment interessante: https://github.com/PrestaShop/PrestaShop/pull/5809 Tu devrais retenter, comme tu connais les biens les themes 1.6, j'aimerais bien avoir ton avis!
×
×
  • Create New...

Important Information

Cookies ensure the smooth running of our services. Using these, you accept the use of cookies. Learn More