Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Catalyst: Dev
Suggested patch to Catalyst::Manual::Tutorial::AdvancedCRUD::FormFu
 

Index | Next | Previous | View Flat


del at alum

Feb 4, 2009, 8:17 AM


Views: 839
Permalink
Suggested patch to Catalyst::Manual::Tutorial::AdvancedCRUD::FormFu

Dear Catalyst developers -

Attached is a patch that updates Catalyst::Manual::Tutorial::AdvancedCRUD::FormFu. I had noticed that both
$form->save_to_model($book) and
$form->defaults_from_model($book)
generated warnings, and that alternatives should be used. E.g.:
save_to_model() method deprecated and is provided for compatibility only, and will be removed: use $form->model->update() instead
I think the patch to the POD reflects the preferred usage. My local tests suggest that it works, but I am still a Catalyst (and FormFu) newbie.

-Del

p.s. - Apologies in advance for the HTML email; I think that otherwise Thunderbird, my MUA, will inline the patch instead of marking it as an attachment.
Attachments: AdvancedCRUD-FormFu.patch (1.90 KB)

Subject User Time
Suggested patch to Catalyst::Manual::Tutorial::AdvancedCRUD::FormFu del at alum Feb 4, 2009, 8:17 AM
    Re: Suggested patch to Catalyst::Manual::Tutorial::AdvancedCRUD::FormFu del at alum Feb 4, 2009, 3:38 PM
        Re: Suggested patch to Catalyst::Manual::Tutorial::AdvancedCRUD::FormFu bobtfish at bobtfish Feb 6, 2009, 7:22 AM

  Index | Next | Previous | View Flat
 
 


Interested in having your list archived? Contact lists@gossamer-threads.com
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.