Giter Site home page Giter Site logo

Comments (3)

crazy-max avatar crazy-max commented on June 20, 2024

Yes j'aurais préféré utiliser composer mais pas le temps de mettre ça en place.
Si t'es motivé j'veux bien :)
Et pourquoi pas mettre ça sur packagist.

from cwsmailbouncehandler.

fpiccinali avatar fpiccinali commented on June 20, 2024

je viens de faire un composer qui fonctionne

je vais faire une pull request dans 5 min.

Il faudrait que tu te créé un compte sur packagist et que tu déclare le projet. Je viens de le faire pour tester avec mon fork et ça a pris 3 minutes. Il ne faudra pas oublier d'activer le webhook dans github pour que les push soient mis à jour.

Il serait jolie de faire un fichier par classes et de les nommer selon la convention PSR qui va bien.
Pour être compatible on pourrait faire des fichiers :
CwsMailBounceHandler.php
CwsMbhCounter.php
CwsMbhMail.php
CwsMbhRecipient.php
CwsMbhResult.php

et dans class.cws.mbh.php on met une rafale d'include.

ainsi l'ancienne manière continuerait de fonctionner et on aurait l'organisation "à la mode"

On pourrait aussi se fendre d'un namespace crazy-max/cws/*

Ou alors on range tout dans un dossier include.

Concernant les class de debug, si tu y tiens vraiment je peux faire un composer pour chacune et mettre les dépendances qu'il faut. Je t'invite toutefois a découvrir le "dump" utilisé par symfony et que l'on peut utiliser indépendemment. Il est très pratique. cf http://symfony.com/doc/current/components/var_dumper/introduction.html

Si ça te conviens, donne moi ton feu vert en fonction de tes décisions et je peux m'en charger.

from cwsmailbouncehandler.

crazy-max avatar crazy-max commented on June 20, 2024

Ok j'ai mergé ton PR et j'avais déjà un compte Packagist mais aucun package mis en place c'est fait maintenant : https://packagist.org/packages/crazy-max/cws-mail-bounce-handler

Sinon oui en effet il faudrait restructurer le projet pour respecter la conv PSR.
Je préférerais plutôt une structure du type tree (Cws/MailBounceHandler/Models/Mail.php, etc...)
Alors oui pour les namespaces et vu qu'il y a un seul service on pourra pour l'instant oublier l'injection de deps bien que plus tard je pourrais refondre cwsDebug et cwsDump (ou bien intégrer Monolog et le varDumper de Symfony comme tu dis par exemple).

A voir donc en tout cas merci pour ta contribution :)

from cwsmailbouncehandler.

Related Issues (18)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.