Page deletion should be moved to the model #87

dandv opened this Issue Aug 13, 2011 · 0 comments

1 participant

MojoMojo member

Right now, page deletion is done in MojoMojo::Controller::Admin::delete(). This checks permissions (a legacy operation since the time the method was located in PageAdmin), and depends on $c for that.

However, there are use cases for deleting pages from outside Catalyst (spam removal, for instance), and the script currently reinvents a poor man's version of page deletion.

The controller method should wrap over a method in the Model that would do the real deletion according to issue #44.

@envi envi added a commit to envi/mojomojo that referenced this issue Aug 27, 2011
@dandv dandv Prepare work on issue #87: move page deletion to the model 97d4bc9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment