Dear Catalyst developers -
While I was working through the Tutorial, I found a couple of nits in,
and also thought I'd add my $0.22 to the guide.
The attached patch represents these changes:
* Tutorial::Authorization - corrects the ACL for "/books/delete" so
that it allows either a "user" or an "admin" to delete books; this
is what is suggested in the text, and prior operation did *not*
allow a non-admin to delete a book. Included here also is a small
editorial comment that allowing users to delete but not create
could be considered an odd bit of policy, but it's just an example.
* Tutorial::AdvancedCRUD::FormFu - purely editorial: suggests how
the intrepid user might now proceed, having completed the Tutorial.
The patch was generated with "svn diff" from the Catalyst-Manual
directory; I hope this is not too inconvenient. I was not willing to
wait for svn's diff over the entire repository.
Apologies in advance if:
1. the diff isn't of the proper format; #catalyst suggested it would
be acceptable
2. the form of attachment is annoying; please let me know how better
to send/attach if this is a problem
3. I've missed some other bit of catalyst-specific patch protocol
-Del
While I was working through the Tutorial, I found a couple of nits in,
and also thought I'd add my $0.22 to the guide.
The attached patch represents these changes:
* Tutorial::Authorization - corrects the ACL for "/books/delete" so
that it allows either a "user" or an "admin" to delete books; this
is what is suggested in the text, and prior operation did *not*
allow a non-admin to delete a book. Included here also is a small
editorial comment that allowing users to delete but not create
could be considered an odd bit of policy, but it's just an example.
* Tutorial::AdvancedCRUD::FormFu - purely editorial: suggests how
the intrepid user might now proceed, having completed the Tutorial.
The patch was generated with "svn diff" from the Catalyst-Manual
directory; I hope this is not too inconvenient. I was not willing to
wait for svn's diff over the entire repository.
Apologies in advance if:
1. the diff isn't of the proper format; #catalyst suggested it would
be acceptable
2. the form of attachment is annoying; please let me know how better
to send/attach if this is a problem
3. I've missed some other bit of catalyst-specific patch protocol
-Del