Ticket #574 (new enhancement)

Opened 8 months ago

Last modified 2 months ago

[PATCH] Ports Page & AbstractModel controllers to RESTful routes

Reported by: bskahan Assigned to:
Priority: major Milestone: 0.6.8 Incise
Component: core Keywords: rest
Cc:

Description

Patch is an intentionally naive refactoring of the pages controller to use RESTful routing. It fully removes the map.connect style named routes, but keeps behavior as unchanged as possible. This basically makes the pages controller "syntactically" RESTful without actually breaking things like "parts", "cache", and "references" into distinct resources.

Based on a quick review, it probably makes sense to port all admin controllers to this level so that AbstractModelController? can be refactored then start on making things "architecturally" RESTful.

All functional tests are updated and passing, added a simple integration test to help the thought process.

Attachments

restful-pages.dif (31.9 kB) - added by bskahan on 11/09/07 04:50:44.
rest_pages_and_abstracts.dif (46.5 kB) - added by bskahan on 11/15/07 00:47:20.

Change History

11/09/07 04:50:44 changed by bskahan

  • attachment restful-pages.dif added.

11/15/07 00:47:20 changed by bskahan

  • attachment rest_pages_and_abstracts.dif added.

11/15/07 00:50:48 changed by bskahan

  • summary changed from [PATCH] Ports 'admin/pages' controller to RESTful routes to [PATCH] Ports Page & AbstractModel controllers to RESTful routes.

Updated to include a pretty thorough transition of Layouts, Snippets, and Users to resource based routing.

11/15/07 12:17:26 changed by seancribbs

  • milestone set to 0.6.6.

05/29/08 17:30:34 changed by airforce1

  • owner changed.
  • component set to core.

thanks favorite air force ones