CircleOfNice / CiRestClientBundle

Mapper for PHP internal curl library
GNU General Public License v3.0
57 stars 20 forks source link

Version 0.2-dev: ErrorCodeMapping #15

Closed TobiasHauck closed 9 years ago

TobiasHauck commented 9 years ago

10

donjuandem commented 9 years ago

gibts irgend en grund dafür, dass du die Werte über default parameter setzt und nicht einfach in der constructor funktion? In meinen Augen gibt es keinen Fall, dass die sich ändern würden (außer die API von CURL ändert sich, aber dann müssen die klassen sowieso angepasst werden, weil, so wie ich das sehe, es ja auch gar nicht vorgesehen ist, die parameter dynamisch zu gestalten, was auch quatsch wäre, wenn du schon eine Millionen Exception classes anlegst :D)

Habe das zufällig nur entdeckt, weil das beim drübergucken aussieht, wie normaler code (->deshalb finde ich generell calls über mehrere Zeilen unlesbar, aber noch schlimmer bei ner definition)

TobiasHauck commented 9 years ago

Je mehr ich darüber nachdenke desto sinnloser ist das. Das was ich damit erreichen wollte haben die Tests erreicht. Was auch mehr Sinn macht :-D

donjuandem commented 9 years ago

rest sieht ok aus, mergen?

TobiasHauck commented 9 years ago

Jup. Merge + neuer Release. Erledige ich gerade.