app/controllers/backend/pages_controller.rb in udongo-2.0.0 vs app/controllers/backend/pages_controller.rb in udongo-2.0.1

@@ -14,36 +14,44 @@
       }
     end
   end
 
   def new
-    @model = Page.new.decorate
+    @form = Backend::PageForm.new(Page.new.decorate)
   end
 
   def create
-    @model = Page.new(params.require('page').permit(:description, :visible, :parent_id)).decorate
+    @form = Backend::PageForm.new(Page.new.decorate)
 
-    if @model.save
-      redirect_to edit_backend_page_path(@model), notice: translate_notice(:added, :page)
+    if @form.save params[:page]
+      redirect_to edit_backend_page_path(@form.page), notice: translate_notice(:added, :page)
     else
       render :new
     end
   end
 
+  def edit
+    @form = Backend::PageForm.new(@model)
+  end
+
   def update
-    if @model.update_attributes(params.require('page').permit(:description, :visible, :parent_id))
+    @form = Backend::PageForm.new(@model)
+
+    if @form.save params[:page]
       redirect_to edit_backend_page_path(@model), notice: translate_notice(:edited, :page)
     else
       render :edit
     end
   end
 
   def tree_drag_and_drop
+    # TODO (Dave) - check if this page is draggable.
     render json: { moved: @model.set_position(params[:position], params[:parent_id]) }
   end
 
   def destroy
+    # TODO (Dave) - check if this page may be destroyed.
     render json: { trashed: @model.destroy }
   end
 
   def page_tree_data(parent_id: nil)
     Page.where(parent_id: parent_id).inject([]) do |data, p|
@@ -59,12 +67,12 @@
     @model = Page.find(params[:id]).decorate
   end
 
   def translation_form
     Backend::PageTranslationForm.new(
-      page: @model,
-      translation: @model.translation(params[:translation_locale]),
-      seo: @model.seo(params[:translation_locale])
+      @model,
+      @model.translation(params[:translation_locale]),
+      @model.seo(params[:translation_locale])
     )
   end
 
   def node_data(page)
     {