Comments (3)
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.
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.
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)
- Bug with eml folder path on Windows HOT 1
- traitment of CwsMbhResult HOT 1
- No way to get mail content and headers? HOT 2
- Bug while processing move operation HOT 1
- MAILBOX_PORT_TLS_SSL 995? HOT 2
- 2 problems HOT 3
- can you make a new stable release HOT 4
- problem with imap_open HOT 4
- Problem with Yahoo HOT 3
- Malformed email can lead to warnings HOT 2
- Bug while creating bounces mailbox HOT 1
- Implement POP3 open mode
- Refactor file open mode HOT 1
- Refactor rule's categories
- Add new rules categories HOT 1
- Implement CwsDebug HOT 1
- Problem processing e-mails HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
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.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from cwsmailbouncehandler.