jakewendt-pages 0.1.2

data/app/views/pages/_page.html.erb

@@ -0,0 +1,35 @@
+<% content_tag_for( :tr, page, :class => 'row' ) do %>
+<td><%= hidden_field_tag( 'pages[]', page.id, :id => nil ) %>
+<%= image_tag 'drag.gif', :class => 'handle' %></td>
+<td class='position'><%= page.position -%></td>
+
+
+<td><table><tbody><tr>
+<td class='path'>
+<%= link_to page.path, 
+	ActionController::Base.relative_url_root.to_s + page.path -%>
+</td>
+<td class='menu'>
+<%= link_to page.menu(session[:locale]), page -%>
+</td>
+<td class='title'>
+<%= link_to page.title(session[:locale]), page -%>
+</td>
+<td class='manage'>
+<% if page.children.length > 1 %>
+<%= link_to "Order", pages_path(:parent_id => page.id) -%>&nbsp;
+<% end %>
+<%= link_to "Edit", edit_page_path(page) -%>&nbsp;
+<%= link_to "Destroy", page_path(page), 
+	:confirm => "Delete '#{page.title(session[:locale])}'?\nAre you sure?", 
+	:method => :delete -%>
+<%#= destroy_link_to "Destroy", page_path(page) -%>
+</td>
+</tr>
+
+<% if page.children.length > 0 %>
+<%= render :partial => 'child', :collection => page.children %>
+<% end %>
+
+</tbody></table></td>
+<% end %><!-- class='page' -->

data/app/views/pages/edit.html.erb

@@ -0,0 +1,9 @@
+<%= error_messages_for :page %>
+
+<% form_for(@page) do |f| %>
+  <%= render 'form', :f => f %>
+  <p>
+    <%= f.submit "Update", :id => nil %>&nbsp;
+    <%= button_link_to "Cancel and View", @page %>
+  </p>
+<% end %>

data/app/views/pages/index.html.erb

@@ -0,0 +1,28 @@
+<% javascripts('pages') %>
+<% stylesheets('pages') %>
+
+<h4>Pages:</h4>
+<% form_tag(order_pages_path, :method => :post, :id => 'order_pages') do -%>
+<%= hidden_field_tag :parent_id, params[:parent_id] %>
+<% unless @pages.empty? %>
+<table id='pages'><tbody>
+<%= render :partial => 'page', :collection => @pages %>
+</tbody></table><!-- id='pages' -->
+<% else %>
+Sorry, no pages yet.
+<% end %>
+
+<p>
+<%= link_to "Create new page", 
+	new_page_path(:parent_id => params[:parent_id]) -%>
+<%= submit_tag 'Save page order', :name => nil, :id => 'save_order' %>
+</p>
+
+<% end %>
+<p>
+The page model is used as a Content Management System 
+and for menu control.  Some routes are not available 
+because the application is using them, but in order to 
+have the menu function, a "fake page" is created as a 
+placeholder.  The staff functionality is an example.
+</p>

data/app/views/pages/new.html.erb

@@ -0,0 +1,9 @@
+<%= error_messages_for :page %>
+
+<% form_for(@page) do |f| %>
+  <%= render 'form', :f => f %>
+  <p>
+    <%= f.submit "Create", :id => nil %>&nbsp;
+		<%= button_link_to "Cancel", pages_path %>
+  </p>
+<% end %>

data/app/views/pages/show.html.erb

@@ -0,0 +1,29 @@
+<% if @page -%>
+<% content_for :head do -%>
+<% javascript_tag do -%>
+if ( typeof(translatables) == 'undefined' ){
+	var translatables = [];
+}
+tmp = { tag: '#page', locales: {} };
+<% %w( en es ).each do |locale| -%>
+tmp.locales["<%=locale%>"]="<%=URI.encode(sanitize(textilize(@page.body(locale))))-%>"
+<% end -%>
+translatables.push(tmp);
+
+tmp = { tag: 'title', locales: {} };
+<% %w( en es ).each do |locale| -%>
+tmp.locales["<%=locale%>"]="<%=URI.encode(@page.title(locale))-%>"
+<% end -%>
+translatables.push(tmp);
+
+tmp = { tag: '#current_root', locales: {} };
+<% %w( en es ).each do |locale| -%>
+tmp.locales["<%=locale%>"]="<%=URI.encode(@page.menu(locale))-%>"
+<% end -%>
+translatables.push(tmp);
+<% end ; end ; end -%>
+<div id='page'>
+<%# if @page -%>
+<%#= sanitize(textilize(@page.body(session[:locale]))) -%>
+<%# end -%>
+</div>

data/app/views/pages/translate.js.erb

@@ -0,0 +1,42 @@
+jQuery(function(){
+
+locale = '<%=session[:locale] || 'en'%>'
+
+set_locale_link = function(l){
+	if( l == 'es' ){
+		locale_link = '<%=link_to( 'English', 
+			locale_path('en'), 
+			:id => 'session_locale' )-%>'
+	}else{
+		locale_link = '<%=link_to( 'Espa&ntilde;ol', 
+			locale_path('es'), 
+			:id => 'session_locale' )-%>'
+	}
+
+	jQuery('#session_locale').replaceWith(locale_link);
+
+	jQuery('#session_locale').click(function(){
+		jQuery.getScript(this.href, function(){
+			translate(locale);
+			set_locale_link(locale);
+		});
+		return false;
+	});
+}
+set_locale_link(locale);
+
+translate = function(l){
+	if( typeof(translatables) != 'undefined' ){
+		jQuery.each(translatables, function(index,value){
+/*
+			jQuery(value.tag).html(decodeURI(value.locales[l]));
+	decodeURI mucked up decoding Entities
+*/
+			jQuery(value.tag).html(decodeURIComponent(value.locales[l]));
+			jQuery(value.tag).fadeIn('slow');	/* pointless */
+		})
+	}
+};
+translate(locale);
+
+});

data/config/routes.rb

@@ -0,0 +1,9 @@
+ActionController::Routing::Routes.draw do |map|
+
+	map.resources :pages, :collection => { 
+		:translate => :get,
+		:order => :post }
+
+	map.resources :locales, :only => :show
+
+end

data/generators/pages/pages_generator.rb

@@ -0,0 +1,66 @@
+class PagesGenerator < Rails::Generator::Base
+
+	def manifest
+		#	See Rails::Generator::Commands::Create
+		#	rails-2.3.10/lib/rails_generator/commands.rb
+		#	for code methods for record (Manifest)
+		record do |m|
+
+			%w( create_pages ).each do |migration|
+				m.migration_template "migrations/#{migration}.rb",
+					'db/migrate', :migration_file_name => migration
+			end
+
+			m.directory('public/javascripts')
+			Dir["#{File.dirname(__FILE__)}/templates/javascripts/*js"].each{|file| 
+				f = file.split('/').slice(-2,2).join('/')
+				m.file(f, "public/javascripts/#{File.basename(file)}")
+			}
+			m.directory('public/stylesheets')
+			Dir["#{File.dirname(__FILE__)}/templates/stylesheets/*css"].each{|file| 
+				f = file.split('/').slice(-2,2).join('/')
+				m.file(f, "public/stylesheets/#{File.basename(file)}")
+			}
+			m.directory('test/functional/pages')
+			Dir["#{File.dirname(__FILE__)}/templates/functional/*rb"].each{|file| 
+				f = file.split('/').slice(-2,2).join('/')
+				m.file(f, "test/functional/pages/#{File.basename(file)}")
+			}
+			m.directory('test/unit/pages')
+			Dir["#{File.dirname(__FILE__)}/templates/unit/*rb"].each{|file| 
+				f = file.split('/').slice(-2,2).join('/')
+				m.file(f, "test/unit/pages/#{File.basename(file)}")
+			}
+		end
+	end
+
+end
+module Rails::Generator::Commands
+	class Create
+		def migration_template(relative_source, 
+				relative_destination, template_options = {})
+			migration_directory relative_destination
+			migration_file_name = template_options[
+				:migration_file_name] || file_name
+			if migration_exists?(migration_file_name)
+				puts "Another migration is already named #{migration_file_name}: #{existing_migrations(migration_file_name).first}: Skipping" 
+			else
+				template(relative_source, "#{relative_destination}/#{next_migration_string}_#{migration_file_name}.rb", template_options)
+			end
+		end
+	end #	Create
+	class Base
+	protected
+		#	the loop through migrations happens so fast
+		#	that they all have the same timestamp which
+		#	won't work when you actually try to migrate.
+		#	All the timestamps MUST be unique.
+		def next_migration_string(padding = 3)
+			@s = (!@s.nil?)? @s.to_i + 1 : if ActiveRecord::Base.timestamped_migrations
+				Time.now.utc.strftime("%Y%m%d%H%M%S")
+			else
+				"%.#{padding}d" % next_migration_number
+			end
+		end
+	end	#	Base
+end

data/generators/pages/templates/functional/locales_controller_test.rb

@@ -0,0 +1,59 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+class Pages::LocalesControllerTest < ActionController::TestCase
+	tests LocalesController
+
+	test "should set locale to en" do
+		assert_nil session[:locale]
+		get :show, :id => 'en'
+		assert_equal 'en', session[:locale]
+		assert_response :redirect
+	end
+
+	test "should change locale to es" do
+		assert_nil session[:locale]
+		session[:locale] = 'en'
+		assert_equal 'en', session[:locale]
+		get :show, :id => 'es'
+		assert_equal 'es', session[:locale]
+		assert_response :redirect
+	end
+
+	test "should change locale to en" do
+		assert_nil session[:locale]
+		session[:locale] = 'es'
+		assert_equal 'es', session[:locale]
+		get :show, :id => 'en'
+		assert_equal 'en', session[:locale]
+		assert_response :redirect
+	end
+
+	test "should set locale to en via js" do
+		@request.accept = "text/javascript"
+		assert_nil session[:locale]
+		get :show, :id => 'en'
+		assert_equal 'en', session[:locale]
+		assert_response :success
+	end
+
+	test "should change locale to es via js" do
+		@request.accept = "text/javascript"
+		assert_nil session[:locale]
+		session[:locale] = 'en'
+		assert_equal 'en', session[:locale]
+		get :show, :id => 'es'
+		assert_equal 'es', session[:locale]
+		assert_response :success
+	end
+
+	test "should change locale to en via js" do
+		@request.accept = "text/javascript"
+		assert_nil session[:locale]
+		session[:locale] = 'es'
+		assert_equal 'es', session[:locale]
+		get :show, :id => 'en'
+		assert_equal 'en', session[:locale]
+		assert_response :success
+	end
+
+end

data/generators/pages/templates/functional/pages_controller_test.rb

@@ -0,0 +1,248 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+class Pages::PagesControllerTest < ActionController::TestCase
+	tests PagesController
+
+	ASSERT_ACCESS_OPTIONS = {
+		:model => 'Page',
+		:actions => [:new,:create,:edit,:update,:destroy,:index],
+		:method_for_create => :factory_create,
+		:attributes_for_create => :factory_attributes
+	}
+
+	def factory_create(options={})
+		Factory(:page,options)
+	end
+	def factory_attributes(options={})
+		Factory.attributes_for(:page,options)
+	end
+
+	assert_access_with_http :show, { :actions => nil }
+	assert_access_with_https :show
+	assert_no_access_with_http 
+
+	assert_access_with_login(:show,{
+		:logins => [:super_user,:admin,:editor]})
+	assert_access_with_login(:show,{
+		:logins => [:interviewer,:reader,:active_user], :actions => nil})
+	assert_access_without_login( :show, { :actions => nil })
+	assert_no_access_with_login({ 
+		:logins => [:interviewer,:reader,:active_user] })
+	assert_no_access_without_login
+
+	assert_no_access_with_login(
+		:attributes_for_create => nil,
+		:method_for_create => nil,
+		:actions => nil,
+		:suffix => " and invalid id",
+		:login => :super_user,
+		:redirect => :pages_path,
+		:edit => { :id => 0 },
+		:update => { :id => 0 },
+		:destroy => { :id => 0 }
+	)
+
+
+%w( super_user admin editor ).each do |cu|
+#
+#		index/new/create/edit/update/destroy 
+#			should only be visible to admins for editing
+#	
+
+	test "should get index with pages with #{cu} login" do
+		3.times{ factory_create }
+		login_as send(cu)
+		get :index
+		assert_template 'index'
+		assert_response :success
+		assert_not_nil assigns(:pages)
+		assigns(:pages).each { |page| assert_nil page.parent }
+	end
+
+	test "should get index with blank parent with #{cu} login" do
+		3.times{ factory_create }
+		login_as send(cu)
+		get :index, :parent_id => ''
+		assert_template 'index'
+		assert_response :success
+		assert_not_nil assigns(:pages)
+		assigns(:pages).each { |page| assert_nil page.parent }
+	end
+
+	test "should get index with subpages with #{cu} login" do
+		parent = factory_create
+		3.times{ factory_create(:parent_id => parent.id) }
+		login_as send(cu)
+		get :index, :parent_id => parent.id
+		assert_template 'index'
+		assert_response :success
+		assert_not_nil assigns(:pages)
+		assigns(:pages).each do |page|
+			assert_equal page.parent_id, parent.id
+		end
+	end
+
+	test "should create page with parent with #{cu} login" do
+		parent = factory_create
+		login_as send(cu)
+		assert_difference('Page.count') do
+			post :create, :page => factory_attributes(
+				:parent_id => parent.id)
+		end
+		assert_equal parent, assigns(:page).parent
+		assert_redirected_to page_path(assigns(:page))
+	end
+
+	test "should NOT create page with invalid page with #{cu} login" do
+		login_as send(cu)
+		assert_no_difference('Page.count') do
+			post :create, :page => {}
+		end
+		assert_template 'new'
+		assert_response :success
+	end
+
+	test "should NOT update page with invalid page with #{cu} login" do
+		login_as send(cu)
+		put :update, :id => factory_create.id, 
+			:page => { :title => "a" }
+		assert_not_nil flash[:error]
+		assert_template 'edit'
+		assert_response :success
+	end
+
+	#	I don't think that this is pertinant anymore
+	test "should get index with both help and non-help pages with #{cu} login" do
+		#	test css menus
+		login_as send(cu)
+		nonhelp_page = factory_create(:path => "/hello" )
+		help_page = factory_create(:path => "/help/test" )
+		get :index
+		assert_response :success
+		assert_template 'index'
+	end
+
+#	action: order
+
+	test "should order pages with #{cu} login" do
+		login_as send(cu)
+#		pages = 3.times.collect{|i| factory_create }
+#	3.times.collect doesn't work on 
+#> ruby --version
+#ruby 1.8.6 (2008-08-11 patchlevel 287) [universal-darwin9.0]
+		pages = []
+		3.times{ pages.push(factory_create) }
+		before_page_ids = Page.all.collect(&:id)
+		post :order, :pages => before_page_ids.reverse
+		after_page_ids = Page.all.collect(&:id)
+		assert_equal after_page_ids, before_page_ids.reverse
+		assert_redirected_to pages_path
+	end
+
+	test "should order sub pages with #{cu} login" do
+		login_as send(cu)
+		parent = factory_create
+		pages = []
+		3.times{ pages.push(factory_create(:parent_id => parent.id)) }
+		assert_equal [1,2,3], pages.collect(&:position)
+		before_page_ids = parent.reload.children.collect(&:id)
+		post :order,:parent_id => parent.id, :pages => before_page_ids.reverse
+		after_page_ids = parent.reload.children.collect(&:id)
+		assert_equal after_page_ids, before_page_ids.reverse
+		assert_redirected_to pages_path(:parent_id => parent.id)
+	end
+
+end
+
+%w( interviewer reader active_user ).each do |cu|
+
+	test "should NOT order pages with #{cu} login" do
+		login_as send(cu)
+		pages = []
+		3.times{ pages.push(factory_create) }
+		before_page_ids = Page.all.collect(&:id)
+		post :order, :pages => before_page_ids.reverse
+		assert_not_nil flash[:error]
+		assert_redirected_to root_path
+	end
+
+end
+
+	test "should NOT order pages without login" do
+		pages = []
+		3.times{ pages.push(factory_create) }
+		before_page_ids = Page.all.collect(&:id)
+		post :order, :pages => before_page_ids.reverse
+		assert_redirected_to_login
+	end
+
+#
+#	/pages/:id should be visible to anyone ?
+#
+#	as the pages controller uses the gateway filter, not being logged in currently
+#	requires the addition of the line ...
+#		CASClient::Frameworks::Rails::GatewayFilter.stubs(:filter).returns(false)
+#
+
+	test "should NOT show page with invalid id" do
+		get :show, :id => 0
+		assert_not_nil flash[:error]
+		assert_template 'show'
+		assert_response :success
+	end
+
+	test "should NOT show page without matching path" do
+		get :show, :path => "/i/do/not/exist".split('/').delete_if{|x|x.blank?}
+		assert_not_nil flash[:error]
+		assert_template 'show'
+		assert_response :success
+	end
+
+	test "should show page by path" do
+		page = factory_create
+		get :show, :path => page.path.split('/').delete_if{|x|x.blank?}
+		assert_equal assigns(:page), page
+		assert_template 'show'
+		assert_response :success
+		assert_select 'title', page.title
+	end
+
+	test "should show page by path with slashes" do
+		page = factory_create(:path => "/help/blogs")
+		get :show, :path => page.path.split('/').delete_if{|x|x.blank?}
+		assert_equal assigns(:page), page
+		assert_template 'show'
+		assert_response :success
+		assert_select 'title', page.title
+	end
+
+	test "should show HOME page without HPP" do
+		page = Page.by_path('/')
+		get :show, :id => page.id
+		assert_nil assigns(:hpp)
+		assert_template 'show'
+		assert_response :success
+		assert_select 'title', page.title
+	end
+
+	test "should get translate via js without login" do
+		@request.accept = "text/javascript"
+		get :translate
+		assert_response :success
+		assert_match /jQuery/, @response.body
+	end
+
+	test "should get each page in fixtures in each locale" do
+		puts
+		Page.all.each do |page|
+			[ nil,'en','es' ].each do |locale|
+				puts "- Showing page id #{page.id} with locale #{locale}"
+				session[:locale] = locale
+				get :show, :id => page.id
+				assert_response :success
+				assert_template 'show'
+			end
+		end
+	end
+
+end