jakewendt-pages 0.1.2

data/generators/pages/templates/javascripts/pages.js

@@ -0,0 +1,39 @@
+var initial_page_order;
+jQuery(function(){
+	jQuery('#pages').sortable({
+		axis:'y', 
+		dropOnEmpty:false, 
+		handle:'img.handle', 
+		update:function(event,ui){compare_page_order()},
+		items:'tr.page.row'
+	});
+
+	jQuery('#save_order').disable();
+
+	initial_page_order = page_order();
+
+	jQuery('form#order_pages').submit(function(){
+		if( initial_page_order == page_order() ) {
+			/*
+				Shouldn't get here as button should 
+				be disabled if not different!
+			*/
+			alert("Page order hasn't changed. Nothing to save.");
+			return false
+		}
+	})
+
+});
+
+function page_order() {
+	return jQuery('#pages').sortable('serialize',{key:'pages[]'});
+}
+
+function compare_page_order(){
+	if( initial_page_order == page_order() ) {
+		jQuery('#save_order').disable();
+	} else {
+		jQuery('#save_order').highlight(4000);
+		jQuery('#save_order').enable();
+	}
+}

data/generators/pages/templates/migrations/create_pages.rb

@@ -0,0 +1,26 @@
+class CreatePages < ActiveRecord::Migration
+	def self.up
+		create_table :pages do |t|
+			t.integer :position
+			t.integer :parent_id
+			t.boolean :hide_menu, :default => false
+			t.string :path
+			t.string :title_en
+			t.string :title_es
+			t.string :menu_en
+			t.string :menu_es
+			t.text :body_en
+			t.text :body_es
+			t.timestamps
+		end
+		add_index :pages, :path, :unique => true
+		add_index :pages, :parent_id
+#	acts_as_list doesn't like the uniqueness
+#	when it reorders, positions are temporarily not unique
+#		add_index :pages, :position, :unique => true, :name => 'by_position'
+	end
+
+	def self.down
+		drop_table :pages
+	end
+end

data/generators/pages/templates/stylesheets/page.css

@@ -0,0 +1,17 @@
+input[type='text'],
+textarea {
+	width: 99%;
+}
+
+div.parent {
+	float:left;
+}
+div.locale {
+	float:left;
+}
+div.hide_menu {
+	float:right;
+}
+div.fields {
+	clear:left;
+}

data/generators/pages/templates/stylesheets/pages.css

@@ -0,0 +1,22 @@
+.page img.handle {
+}
+.page .position {
+	font-weight:bold;
+	text-align:center;
+}
+.page .path {
+	width: 140px;
+}
+.page .menu {
+	width: 140px;
+}
+.page .title {
+	width: 140px;
+}
+.page .manage {
+	text-align:right;
+}
+
+.page.row.ui-sortable-helper {
+	border: 1px dashed #E7C254;
+}

data/generators/pages/templates/unit/page_test.rb

@@ -0,0 +1,185 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+class Pages::PageTest < ActiveSupport::TestCase
+
+	assert_should_require(:path,:menu_en,:title_en,:body_en,
+		:model => 'Page')
+	assert_should_require_unique(:path,:menu_en,
+		:model => 'Page')
+
+	test "should create page" do
+		assert_difference 'Page.count' do
+			page = create_page
+			assert !page.new_record?, 
+				"#{page.errors.full_messages.to_sentence}"
+		end
+	end
+
+	test "should require path begin with slash" do
+		assert_no_difference 'Page.count' do
+			page = create_page(:path => 'Hey')
+			assert page.errors.on(:path)
+		end
+	end
+
+	test "should require 1 char path" do
+		assert_no_difference 'Page.count' do
+			page = create_page(:path => '')
+			assert page.errors.on(:path)
+		end
+	end
+
+	test "should require 4 char menu_en" do
+		assert_no_difference 'Page.count' do
+			page = create_page(:menu_en => 'Hey')
+			assert page.errors.on(:menu_en)
+		end
+	end
+
+	test "should require 4 char title_en" do
+		assert_no_difference 'Page.count' do
+			page = create_page(:title_en => 'Hey')
+			assert page.errors.on(:title_en)
+		end
+	end
+
+	test "should require 4 char body_en" do
+		assert_no_difference 'Page.count' do
+			page = create_page(:body_en => 'Hey')
+			assert page.errors.on(:body_en)
+		end
+	end
+
+	test "should filter out multiple continguous slashes" do
+		page = create_page(:path => "///a//b///c" )
+		assert_equal "/a/b/c", page.path
+	end
+
+	test "should downcase path" do
+		page = create_page(:path => "/A/B/C")
+		assert_equal "/a/b/c", page.path
+	end
+
+	test "can have a parent" do
+		parent = create_page
+		page = create_page( :parent_id => parent.id )
+		assert_equal page.reload.parent, parent
+	end
+
+	test "should return self as root with no parent" do
+		page = create_page
+		assert_equal page, page.root
+	end
+
+	test "should return parent as root with parent" do
+		parent = create_page
+		page = create_page( :parent_id => parent.id )
+		assert_equal parent, page.reload.root
+	end
+
+	test "should nullify parent_id of children when parent destroyed" do
+		parent = create_page
+		child  = create_page( :parent_id => parent.id )
+		assert_equal child.reload.parent_id, parent.id
+		parent.destroy
+		assert_nil child.reload.parent_id
+	end
+
+	test "should return false if page is not home" do
+		page = create_page
+		assert !page.is_home?
+	end
+
+	test "should return true if page is home" do
+		page = create_page(:path => '/')
+		assert page.is_home?
+	end
+
+	test "should create page with hide_menu true" do
+		assert_difference('Page.count',1){
+		assert_difference('Page.roots.count',0){
+			page = create_page(:hide_menu => true)
+#			assert_equal 1, Page.count
+#			assert_equal 0, Page.roots.count
+			assert_not_nil Page.find(page)
+			assert_not_nil Page.find(page.id)
+			assert_not_nil Page.find_by_path(page.path)
+		} }
+	end
+
+	test "should find page by path" do
+		p = create_page
+		page = Page.by_path(p.path)
+		assert_equal p, page
+	end
+
+	test "should return english menu without locale" do
+		p = create_page
+		assert_equal p.menu, p.menu_en
+	end
+
+	test "should return english title without locale" do
+		p = create_page
+		assert_equal p.title, p.title_en
+	end
+
+	test "should return english body without locale" do
+		p = create_page
+		assert_equal p.body, p.body_en
+	end
+
+	test "should return english menu with locale" do
+		p = create_page
+		assert_equal p.menu('en'), p.menu_en
+	end
+
+	test "should return english title with locale" do
+		p = create_page
+		assert_equal p.title('en'), p.title_en
+	end
+
+	test "should return english body with locale" do
+		p = create_page
+		assert_equal p.body('en'), p.body_en
+	end
+
+	test "should return spanish menu with locale" do
+		p = create_page(:menu_es => 'spanish menu')
+		assert_equal p.menu('es'), p.menu_es
+	end
+
+	test "should return spanish title with locale" do
+		p = create_page(:title_es => 'spanish title')
+		assert_equal p.title('es'), p.title_es
+	end
+
+	test "should return spanish body with locale" do
+		p = create_page(:body_es => 'spanish body')
+		assert_equal p.body('es'), p.body_es
+	end
+
+	test "should return english menu with missing spanish locale" do
+		p = create_page(:menu_es => '')
+		assert_equal p.menu('es'), p.menu_en
+	end
+
+	test "should return english title with missing spanish locale" do
+		p = create_page(:title_es => '')
+		assert_equal p.title('es'), p.title_en
+	end
+
+	test "should return english body with missing spanish locale" do
+		p = create_page(:body_es => '')
+		assert_equal p.body('es'), p.body_en
+	end
+
+protected
+
+	def create_page(options = {})
+		record = Factory.build(:page,options)
+		record.save
+		record
+	end
+	alias_method :create_object, :create_page
+
+end

data/generators/pages/templates/unit/redcloth_extension_test.rb

@@ -0,0 +1,64 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+class Pages::RedClothExtensionTest < ActiveSupport::TestCase
+
+	teardown :destroy_relative_url_root
+
+	test "should NOT add relative url root to link when blank" do
+		html = textilize('"Link Text":/link/path')
+		#=> "<p><a href=\"/link/path\">Link Text</a></p>"
+		assert_match(/a href="\/link\/path"/,html)
+	end
+
+	test "should add relative url root to link when not blank" do
+		ActionController::Base.relative_url_root = "/prefix"
+		html = textilize('"Link Text":/link/path')
+		#=> "<p><a href=\"/prefix/link/path\">Link Text</a></p>"
+		assert_match(/a href="\/prefix\/link\/path"/,html)
+	end
+
+	test "should NOT add relative url root to image href when blank" do
+		html = textilize('!/path/to/image.jpg!:/link/path')
+		#=> "<p><a href=\"/link/path\">
+		#	<img src=\"/path/to/image.jpg\" alt=\"\" /></a></p>"
+		assert_match(/a href="\/link\/path"/,html)
+	end
+
+	test "should add relative url root to image href when not blank" do
+		ActionController::Base.relative_url_root = "/prefix"
+		html = textilize('!/path/to/image.jpg!:/link/path')
+		#=> "<p><a href=\"/link/path\">
+		#	<img src=\"/path/to/image.jpg\" alt=\"\" /></a></p>"
+		assert_match(/a href="\/prefix\/link\/path"/,html)
+	end
+
+	test "should NOT add relative url root to image src when blank" do
+		html = textilize('!/path/to/image.jpg!:/link/path')
+		#=> "<p><a href=\"/link/path\">
+		#	<img src=\"/path/to/image.jpg\" alt=\"\" /></a></p>"
+		assert_match(/img src="\/path\/to\/image\.jpg"/,html)
+	end
+
+	test "should add relative url root to image src when not blank" do
+		ActionController::Base.relative_url_root = "/prefix"
+		html = textilize('!/path/to/image.jpg!:/link/path')
+		#=> "<p><a href=\"/link/path\">
+		#	<img src=\"/path/to/image.jpg\" alt=\"\" /></a></p>"
+		assert_match(/img src="\/prefix\/path\/to\/image\.jpg"/,html)
+	end
+
+
+#	Add tests for when 
+#		relative_url_root is nil
+#		paths are absolute
+#		paths are relative to current location ('../......')
+#		there is no path, just the file
+
+
+protected
+
+	def destroy_relative_url_root
+		ActionController::Base.relative_url_root = nil	#''
+	end
+
+end

data/lib/pages.rb

@@ -0,0 +1,42 @@
+require 'ruby_extension'
+require 'rails_helpers'
+require 'authorized'
+require 'gravatar'
+require 'calnet_authenticated'
+require 'acts_as_list'
+module Pages
+#	predefine namespace
+end
+
+#	This doesn't seem necessary
+%w{models controllers}.each do |dir|
+	path = File.expand_path(File.join(File.dirname(__FILE__), '../app', dir))
+	ActiveSupport::Dependencies.autoload_paths << path
+	ActiveSupport::Dependencies.autoload_once_paths << path
+end
+
+HTML::WhiteListSanitizer.allowed_attributes.merge(%w(
+	id class style
+))
+
+require 'pages/redcloth/formatters/html'
+
+if !defined?(RAILS_ENV) || RAILS_ENV == 'test'
+	require 'active_support'
+	require 'active_support/test_case'
+	require 'factory_girl'
+	require 'assert_this_and_that'
+	require 'pages/factories'
+	require 'pages/pending'
+end
+
+ActionController::Routing::Routes.add_configuration_file(
+	File.expand_path(
+		File.join(
+			File.dirname(__FILE__), '../config/routes.rb')))
+
+ActionController::Base.view_paths <<
+	File.expand_path(
+		File.join(
+			File.dirname(__FILE__), '../app/views'))
+

data/lib/pages/factories.rb

@@ -0,0 +1,6 @@
+Factory.define :page do |f|
+	f.sequence(:path) { |n| "/path#{n}" }
+	f.sequence(:menu_en) { |n| "Menu #{n}" }
+	f.sequence(:title_en){ |n| "Title #{n}" }
+	f.body_en  "Page Body"
+end

data/lib/pages/pending.rb

@@ -0,0 +1,72 @@
+# Some code from jeremymcanally's "pending"
+# http://github.com/jeremymcanally/pending/tree/master
+
+module ActiveSupport
+	module Testing
+		module Pending
+
+			unless defined?(Spec)
+
+				@@pending_cases = []
+				@@at_exit = false
+
+				def pending(description = "", &block)
+					if description.is_a?(Symbol)
+						is_pending = $tags[description]
+						return block.call unless is_pending
+					end
+
+					if block_given?
+						failed = false
+
+						begin
+							block.call
+						rescue Exception
+							failed = true
+						end
+
+						flunk("<#{description}> did not fail.") unless failed 
+					end
+
+					caller[0] =~ (/(.*):(.*):in `(.*)'/)
+#puts caller.inspect
+
+# looks like we lose the name of the 'method' in 1.9.1
+#"/Users/jakewendt/github_repo/jakewendt/ucb_ccls_homex/test/unit/subject_test.rb:145:in `block in <class:SubjectTest>'", 
+
+#					@@pending_cases << "#{$3} at #{$1}, line #{$2}"
+					#	Gotta remember these as the next Regex will overwrite them.
+					filename   = $1
+					linenumber = $2
+#	ruby 1.8.7
+#	Hx/Addresses Controller should NOT create new address with employee login and invalid address:
+#	ruby 1.9.1
+#Hx/Addresses Controller block (2 levels) in <class:AddressesControllerTest>:
+					testmethod = $3
+
+					model  = self.class.to_s.gsub(/Test$/,'').titleize
+					method = testmethod.gsub(/_/,' ').gsub(/^test /,'')
+					@@pending_cases << "#{model} #{method}:\n.\t#{filename} line #{linenumber}"
+#					@@pending_cases << "#{testmethod} at #{filename}, line #{linenumber}"
+					print "P"
+			
+					@@at_exit ||= begin
+						at_exit do
+							#	For some reason, special characters don't always
+							#	print the way you would expect.  Leading white space (tabs)
+							#	and some carriage returns just weren't happening?
+							#	Is this at_exit doing some parsing??
+							puts "\nPending Cases:"
+							@@pending_cases.each do |test_case|
+								puts test_case
+							end
+							puts " \n"
+						end
+					end
+				end
+			end
+			
+		end
+	end
+end
+ActiveSupport::TestCase.send(:include, ActiveSupport::Testing::Pending)