From 7f6133d27f8a3edab86403a6f5bc4f815525de42 Mon Sep 17 00:00:00 2001
From: Josh Adams <jadams@quadtopus.(none)>
Date: Thu, 12 Mar 2009 13:10:25 -0500
Subject: [PATCH] fixed some conflicts from a cherry-pick

---
 app/controllers/application.rb      |  1 -
 app/controllers/page_controller.rb  |  2 +-
 app/views/page/_breadcrumb.html.erb | 16 +++++++---------
 3 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/app/controllers/application.rb b/app/controllers/application.rb
index fea4cc4..b8f3488 100644
--- a/app/controllers/application.rb
+++ b/app/controllers/application.rb
@@ -19,7 +19,6 @@ def set_layout_variables
       @topNavChildren[sub] = temp if temp.length > 0
     end
     @topNav.unshift @root
-    @breadcrumb = []
   end
 
   def load_menu
diff --git a/app/controllers/page_controller.rb b/app/controllers/page_controller.rb
index 70d2ecf..ee42221 100644
--- a/app/controllers/page_controller.rb
+++ b/app/controllers/page_controller.rb
@@ -4,7 +4,7 @@ class PageController < ApplicationController
   # Currently in need of some serious refactoring
   def indexer
     @page = Page.visible.find_page_by_path params['path']
-    @breadcrumb = @page.ancestors.reverse.to_a if @page
+    @breadcrumb = @page.ancestors.reverse.collect { |x| [x.name, x.title] } if @page
     @top_page = params[:path][0] || 'root'
     render :template => "404", :status => 404, :layout => false and return unless @page
     if @page.is_published? || (logged_in? && current_user.can_view_drafts?)
diff --git a/app/views/page/_breadcrumb.html.erb b/app/views/page/_breadcrumb.html.erb
index 9b552a3..967ce03 100644
--- a/app/views/page/_breadcrumb.html.erb
+++ b/app/views/page/_breadcrumb.html.erb
@@ -1,10 +1,8 @@
-<div id="breadcrumb">
-  <% url = ""
-    @breadcrumb.each do |crumb| 
-      url = (url != "") ? url + crumb.name + "/" : url = "/" %>
-  <%= link_to crumb.title, ((url=='/')? '/pages/' + @root.name + '/' : url ) %> &gt; 
-  <% end %>
-  <% if @page -%>
-    <b><%= (@pageNum!=nil)? link_to(@page.title, './') + " &gt; page " + @pageNum.to_s : @page.title %></b>
-  <% end -%>
+<div id="breadcrumb">
+  <% @breadcrumb.each do |crumb| %>
+    <a href="/<%= crumb[0] -%>"><%= crumb[1] -%></a>&nbsp;&gt;&nbsp;
+  <% end %>
+  <% if @page -%>
+    <b><%= (@pageNum!=nil)? link_to(@page.title, './') + " &gt; page " + @pageNum.to_s : @page.title %></b>
+  <% end -%>
 </div>