On Apr 26, 2008, at 12:10 AM, Thomas R. Koll wrote:
Am 26.04.2008 um 00:07 schrieb Phillip Oertel:
achso!
rest-mäßig erstellst du ein neues objekt, also sollte es ein POST
auf die collection des models werden.
lass doch den request auf create gehen, und gib beim klicken dex
(+) die id des zu kopierenden datensatzes mit (z.b. als
"create_from_id"). in der create-methode wird dann unterschieden ob
die create_from_id mitkommt, wenn ja wird der datensatz "geklont"
und mit geändertem datum neu gespeichert, wenn nein wird ein
normaler create. dagegen ist imho nichts einzuwenden.
Sorry, aber das ist einfach nur dreckiger Code und verwirrt auch
schnell den nächsten Programmierer.
Wieso das? Bei dieser Lösung finde ich den Code zum Erzeugen in
create, d.h. dort, wo ich ihn erwarte. Erscheint mir trotz des if-
statements genau so gültig wir die Fallunterscheidung per routes.rb.
Ist ansonsten wahrscheinlich Geschmackssache. REST-mäßig wäre das Neu-
Anlegen in diesem Kontext auf jeden Fall ein POST, nicht ein PUT.
Viele Grüße,
Stefan
--
Stefan Tilkov, http://www.innoq.com/blog/st/
eine zusätzliche Methode lässt sich viel schneller finden und schon
allein vom Namen verstehen.
Man spart ja auch die lästigen Kommentare damit ;-)
ciao, tom
--
Thomas R. "TomK32" Koll || http://tomk32.de || http://ananasblau.com
just a geek trying to change the world
Skype: TomK32 || Mail: [EMAIL PROTECTED]
_______________________________________________
rubyonrails-ug mailing list
[email protected]
http://mailman.headflash.com/mailman/listinfo/rubyonrails-ug
_______________________________________________
rubyonrails-ug mailing list
[email protected]
http://mailman.headflash.com/mailman/listinfo/rubyonrails-ug
