search

youngguns-nl / moneybird_php_api

A PHP Library for the MoneyBird API
http://www.moneybird.nl
MIT License
37 stars 14 forks source link

Namespacing volgens PSR-0 #14

Closed pluijm closed 11 years ago

pluijm commented 12 years ago

"Lijkt me mooier om PSR-0 aan te houden zodat Moneybird/Error/Array linkt naar Error/Array.php."

ruudk commented 12 years ago

https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-0.md

Wordt gebruikt door grote frameworks als Zend Framework2, Symfony2, etc...

pluijm commented 12 years ago

Zoals ik het nu lees (en toen ik hieraan begon ook), is het toepassen van namespacing op het niveau dat jij noemt een kwestie van smaak. Zoals ik het nu heb toegepast is het volgens PSR-0, maar heb ik de namespacing niet verder doorgevoerd.

ruudk commented 12 years ago

Hmm, dus je gaat het niet aanpassen :( ?

pluijm commented 12 years ago

Zeg nooit nooit :) Ik ga het in ieder geval niet meteen oppakken

ruudk commented 11 years ago

Merge je het als ik de PR aanlever? Dan wil ik het wel doen namelijk.

pluijm commented 11 years ago

Dat ligt eraan hoe het geïmplementeerd is. Als ik me erin kan vinden natuurlijk wel. Misschien is het een idee om het eerst even droog uit te werken en voor te stellen?

pluijm commented 11 years ago

Ben er aan begonnen, zal ergens in juli doorgevoerd worden

ruudk commented 11 years ago

:+1: Thanks!