Merge branch 'master' into move-article-folder

Conflicts: app/helpers/forms_helper.rb

Merge branch 'master' into move-article-folder
Conflicts: app/helpers/forms_helper.rb
Rodrigo Souto
2 parents febc53cd f1e0dc83
Showing 159 changed files with 5612 additions and 622 deletions Show diff stats
app/controllers/application_controller.rb
app/controllers/box_organizer_controller.rb
app/controllers/my_profile/profile_design_controller.rb
app/controllers/public/account_controller.rb
app/controllers/public/content_viewer_controller.rb
app/helpers/application_helper.rb
app/helpers/boxes_helper.rb
app/helpers/cms_helper.rb
app/helpers/colorbox_helper.rb
app/helpers/forms_helper.rb
app/models/box.rb
app/models/comment.rb
app/models/person.rb
app/models/task.rb
app/models/user.rb
app/views/account/login.rhtml
app/views/account/login_block.rhtml
app/views/box_organizer/_block_types.rhtml
app/views/box_organizer/_highlights_block.rhtml
app/views/box_organizer/_link_list_block.rhtml
@@ -114,7 +114,9 @@ class ApplicationController &lt; ActionController::Base
   # plugin to the current controller being initialized.
   def init_noosfero_plugins_controller_filters
     plugins.each do |plugin|
-      plugin.send(self.class.name.underscore + '_filters').each do |plugin_filter|
+      filters = plugin.send(self.class.name.underscore + '_filters')
+      filters = [filters] if !filters.kind_of?(Array)
+      filters.each do |plugin_filter|
         self.class.send(plugin_filter[:type], plugin.class.name.underscore + '_' + plugin_filter[:method_name], (plugin_filter[:options] || {}))
         self.class.send(:define_method, plugin.class.name.underscore + '_' + plugin_filter[:method_name], plugin_filter[:block])
       end
@@ -68,7 +68,8 @@ class BoxOrganizerController &lt; ApplicationController
         raise ArgumentError.new("Type %s is not allowed. Go away." % type)
       end
     else
-      @block_types = available_blocks
+      @center_block_types = Box.acceptable_center_blocks & available_blocks
+      @side_block_types = Box.acceptable_side_blocks & available_blocks
       @boxes = boxes_holder.boxes
       render :action => 'add_block', :layout => false
     end
@@ -5,7 +5,7 @@ class ProfileDesignController &lt; BoxOrganizerController
   protect 'edit_profile_design', :profile
   def available_blocks
-    blocks = [ ArticleBlock, TagsBlock, RecentDocumentsBlock, ProfileInfoBlock, LinkListBlock, MyNetworkBlock, FeedReaderBlock, ProfileImageBlock, LocationBlock, SlideshowBlock, ProfileSearchBlock ]
+    blocks = [ ArticleBlock, TagsBlock, RecentDocumentsBlock, ProfileInfoBlock, LinkListBlock, MyNetworkBlock, FeedReaderBlock, ProfileImageBlock, LocationBlock, SlideshowBlock, ProfileSearchBlock, HighlightsBlock ]
     # blocks exclusive for organizations
     if profile.has_members?
@@ -25,11 +25,13 @@ class AccountController &lt; ApplicationController
   # action to perform login to the application
   def login
-    @user = User.new
-    @person = @user.build_person
     store_location(request.referer) unless session[:return_to]
     return unless request.post?
-    self.current_user = User.authenticate(params[:user][:login], params[:user][:password], environment) if params[:user]
+
+    self.current_user = plugins_alternative_authentication
+
+    self.current_user ||= User.authenticate(params[:user][:login], params[:user][:password], environment) if params[:user]
+
     if logged_in?
       if params[:remember_me] == "1"
         self.current_user.remember_me
@@ -41,7 +43,6 @@ class AccountController &lt; ApplicationController
       end
     else
       session[:notice] = _('Incorrect username or password') if redirect?
-      redirect_to :back if redirect?
     end
   end
@@ -56,6 +57,11 @@ class AccountController &lt; ApplicationController
   # action to register an user to the application
   def signup
+    if @plugins.dispatch(:allow_user_registration).include?(false)
+      redirect_back_or_default(:controller => 'home')
+      session[:notice] = _("This environment doesn't allow user registration.")
+    end
+
     @invitation_code = params[:invitation_code]
     begin
       if params[:user]
@@ -125,6 +131,10 @@ class AccountController &lt; ApplicationController
   #
   # Posts back.
   def forgot_password
+    if @plugins.dispatch(:allow_password_recovery).include?(false)
+      redirect_back_or_default(:controller => 'home')
+      session[:notice] = _("This environment doesn't allow password recovery.")
+    end
     @change_password = ChangePassword.new(params[:change_password])
     if request.post?
@@ -316,4 +326,13 @@ class AccountController &lt; ApplicationController
     end
   end
+  def plugins_alternative_authentication
+    user = nil
+    @plugins.each do |plugin|
+      user = plugin.alternative_authentication
+      break unless user.nil?
+    end
+    user
+  end
+
 end
@@ -2,6 +2,8 @@ class ContentViewerController &lt; ApplicationController
   needs_profile
+  before_filter :comment_author, :only => :edit_comment
+
   helper ProfileHelper
   helper TagsHelper
@@ -121,6 +123,27 @@ class ContentViewerController &lt; ApplicationController
     end
   end
+  def edit_comment
+    path = params[:page].join('/')
+    @page = profile.articles.find_by_path(path)
+    @form_div = 'opened'
+    @comment = @page.comments.find_by_id(params[:id])
+    if @comment
+      if request.post?
+        begin
+          @comment.update_attributes(params[:comment])
+          session[:notice] = _('Comment succesfully updated')
+          redirect_to :action => 'view_page', :profile => profile.identifier, :page => @comment.article.explode_path
+        rescue
+          session[:notice] = _('Comment could not be updated')
+        end
+      end
+    else
+      redirect_to @page.view_url
+      session[:notice] = _('Could not find the comment in the article')
+    end
+  end
+
   protected
   def add_comment
@@ -198,4 +221,13 @@ class ContentViewerController &lt; ApplicationController
     end
   end
+  def comment_author
+    comment = Comment.find_by_id(params[:id])
+    if comment
+      render_access_denied if comment.author.blank? || comment.author != user
+    else
+      render_not_found
+    end
+  end
+
 end
@@ -265,9 +265,9 @@ module ApplicationHelper
   VIEW_EXTENSIONS = %w[.rhtml .html.erb]
-  def partial_for_class_in_view_path(klass, view_path)
+  def partial_for_class_in_view_path(klass, view_path, suffix = nil)
     return nil if klass.nil?
-    name = klass.name.underscore
+    name = [klass.name.underscore, suffix].compact.map(&:to_s).join('_')
     search_name = String.new(name)
     if search_name.include?("/")
@@ -285,28 +285,17 @@ module ApplicationHelper
     partial_for_class_in_view_path(klass.superclass, view_path)
   end
-  def partial_for_class(klass)
+  def partial_for_class(klass, suffix=nil)
     raise ArgumentError, 'No partial for object. Is there a partial for any class in the inheritance hierarchy?' if klass.nil?
     name = klass.name.underscore
     @controller.view_paths.each do |view_path|
-      partial = partial_for_class_in_view_path(klass, view_path)
+      partial = partial_for_class_in_view_path(klass, view_path, suffix)
       return partial if partial
     end
     raise ArgumentError, 'No partial for object. Is there a partial for any class in the inheritance hierarchy?'
   end
-  def partial_for_task_class(klass, action)
-    raise ArgumentError, 'No partial for object. Is there a partial for any class in the inheritance hierarchy?' if klass.nil?
-
-    name = "#{klass.name.underscore}_#{action.to_s}"
-    VIEW_EXTENSIONS.each do |ext|
-      return name if File.exists?(File.join(RAILS_ROOT, 'app', 'views', params[:controller], '_'+name+ext))
-    end
-
-    partial_for_task_class(klass.superclass, action)
-  end
-
   def view_for_profile_actions(klass)
     raise ArgumentError, 'No profile actions view for this class.' if klass.nil?
@@ -1323,6 +1312,19 @@ module ApplicationHelper
     end
   end
+  def expirable_link_to(expired, content, url, options = {})
+    if expired
+      options[:class] = (options[:class] || '') + ' disabled'
+      content_tag('a', '&nbsp;'+content_tag('span', content), options)
+    else
+      link_to content, url, options
+    end
+  end
+
+  def remove_content_button(action)
+    @plugins.dispatch("content_remove_#{action.to_s}", @page).include?(true)
+  end
+
   def template_options(klass, field_name)
     return '' if klass.templates.count == 0
     return hidden_field_tag("#{field_name}[template_id]", klass.templates.first.id) if klass.templates.count == 1
@@ -1388,4 +1390,19 @@ module ApplicationHelper
     result
   end
+  def expirable_content_reference(content, action, text, url, options = {})
+    reason = @plugins.dispatch("content_expire_#{action.to_s}", content).first
+    options[:title] = reason
+    expirable_link_to reason.present?, text, url, options
+  end
+
+  def expirable_button(content, action, text, url, options = {})
+    options[:class] = "button with-text icon-#{action.to_s}"
+    expirable_content_reference content, action, text, url, options
+  end
+
+  def expirable_comment_link(content, action, text, url, options = {})
+    options[:class] = "comment-footer comment-footer-link comment-footer-hide"
+    expirable_content_reference content, action, text, url, options
+  end
 end
@@ -162,9 +162,6 @@ module BoxesHelper
   #
   # +box+ is always needed
   def block_target(box, block = nil)
-    # FIXME hardcoded
-    return '' if box.position == 1
-
     id =
       if block.nil?
         "end-of-box-#{box.id}"
@@ -172,14 +169,11 @@ module BoxesHelper
         "before-block-#{block.id}"
       end
-    content_tag('div', '&nbsp;', :id => id, :class => 'block-target' ) + drop_receiving_element(id, :url => { :action => 'move_block', :target => id }, :accept => 'block', :hoverclass => 'block-target-hover')
+    content_tag('div', '&nbsp;', :id => id, :class => 'block-target' ) + drop_receiving_element(id, :url => { :action => 'move_block', :target => id }, :accept => box.acceptable_blocks, :hoverclass => 'block-target-hover')
   end
   # makes the given block draggable so it can be moved away.
   def block_handle(block)
-    # FIXME hardcoded
-    return '' if block.box.position == 1
-
     draggable_element("block-#{block.id}", :revert => true)
   end
@@ -211,7 +205,7 @@ module BoxesHelper
     end
     if block.editable?
-      buttons << lightbox_icon_button(:edit, _('Edit'), { :action => 'edit', :id => block.id })
+      buttons << colorbox_icon_button(:edit, _('Edit'), { :action => 'edit', :id => block.id })
     end
     if !block.main?
@@ -42,13 +42,25 @@ module CmsHelper
   def display_spread_button(profile, article)
     if profile.person?
-      button_without_text :spread, _('Spread this'), :action => 'publish', :id => article.id
+      expirable_button article, :spread, _('Spread this'), :action => 'publish', :id => article.id
     elsif profile.community? && environment.portal_community
-      button_without_text :spread, _('Spread this'), :action => 'publish_on_portal_community', :id => article.id
+      expirable_button article, :spread, _('Spread this'), :action => 'publish_on_portal_community', :id => article.id
     end
   end
   def display_delete_button(article)
-    button_without_text :delete, _('Delete'), { :action => 'destroy', :id => article.id }, :method => :post, :confirm => delete_article_message(article)
+    expirable_button article, :delete, _('Delete'), { :action => 'destroy', :id => article.id }, :method => :post, :confirm => delete_article_message(article)
+  end
+
+  def expirable_button(content, action, title, url, options = {})
+    reason = @plugins.dispatch("content_expire_#{action.to_s}", content).first
+    if reason.present?
+      options[:class] = (options[:class] || '') + ' disabled'
+      options[:disabled] = 'disabled'
+      options.delete(:confirm)
+      options.delete(:method)
+      title = reason
+    end
+    button_without_text action.to_sym, title, url, options
   end
 end
@@ -8,6 +8,10 @@ module ColorboxHelper
     button(type, label, url, colorbox_options(options))
   end
+  def colorbox_icon_button(type, label, url, options = {})
+    icon_button(type, label, url, colorbox_options(options))
+  end
+
   # options must be an HTML options hash as passed to link_to etc.
   #
   # returns a new hash with colorbox class added. Keeps existing classes.
@@ -155,6 +155,119 @@ module FormsHelper
     return result
   end
+  def date_field(name, value, format = '%Y-%m-%d', datepicker_options = {}, html_options = {})
+    datepicker_options[:disabled] ||= false
+    datepicker_options[:alt_field] ||= ''
+    datepicker_options[:alt_format] ||= ''
+    datepicker_options[:append_text] ||= ''
+    datepicker_options[:auto_size] ||= false
+    datepicker_options[:button_image] ||= ''
+    datepicker_options[:button_image_only] ||=  false
+    datepicker_options[:button_text] ||= '...'
+    datepicker_options[:calculate_week] ||= 'jQuery.datepicker.iso8601Week'
+    datepicker_options[:change_month] ||= false
+    datepicker_options[:change_year] ||= false
+    datepicker_options[:close_text] ||= _('Done')
+    datepicker_options[:constrain_input] ||= true
+    datepicker_options[:current_text] ||= _('Today')
+    datepicker_options[:date_format] ||= 'mm/dd/yy'
+    datepicker_options[:day_names] ||= [_('Sunday'), _('Monday'), _('Tuesday'), _('Wednesday'), _('Thursday'), _('Friday'), _('Saturday')]
+    datepicker_options[:day_names_min] ||= [_('Su'), _('Mo'), _('Tu'), _('We'), _('Th'), _('Fr'), _('Sa')]
+    datepicker_options[:day_names_short] ||= [_('Sun'), _('Mon'), _('Tue'), _('Wed'), _('Thu'), _('Fri'), _('Sat')]
+    datepicker_options[:default_date] ||= nil
+    datepicker_options[:duration] ||= 'normal'
+    datepicker_options[:first_day] ||= 0
+    datepicker_options[:goto_current] ||= false
+    datepicker_options[:hide_if_no_prev_next] ||= false
+    datepicker_options[:is_rtl] ||= false
+    datepicker_options[:max_date] ||= nil
+    datepicker_options[:min_date] ||= nil
+    datepicker_options[:month_names] ||= [_('January'), _('February'), _('March'), _('April'), _('May'), _('June'), _('July'), _('August'), _('September'), _('October'), _('November'), _('December')]
+    datepicker_options[:month_names_short] ||= [_('Jan'), _('Feb'), _('Mar'), _('Apr'), _('May'), _('Jun'), _('Jul'), _('Aug'), _('Sep'), _('Oct'), _('Nov'), _('Dec')]
+    datepicker_options[:navigation_as_date_format] ||= false
+    datepicker_options[:next_text] ||= _('Next')
+    datepicker_options[:number_of_months] ||= 1
+    datepicker_options[:prev_text] ||= _('Prev')
+    datepicker_options[:select_other_months] ||= false
+    datepicker_options[:short_year_cutoff] ||= '+10'
+    datepicker_options[:show_button_panel] ||= false
+    datepicker_options[:show_current_at_pos] ||= 0
+    datepicker_options[:show_month_after_year] ||= false
+    datepicker_options[:show_on] ||= 'focus'
+    datepicker_options[:show_options] ||= {}
+    datepicker_options[:show_other_months] ||= false
+    datepicker_options[:show_week] ||= false
+    datepicker_options[:step_months] ||= 1
+    datepicker_options[:week_header] ||= _('Wk')
+    datepicker_options[:year_range] ||= 'c-10:c+10'
+    datepicker_options[:year_suffix] ||= ''
+
+    element_id = html_options[:id] || 'datepicker-date'
+    value = value.strftime(format) if value.present?
+    method = datepicker_options[:time] ? 'datetimepicker' : 'datepicker'
+    result = text_field_tag(name, value, html_options)
+    result +=
+    "
+    <script type='text/javascript'>
+      jQuery('##{element_id}').#{method}({
+        disabled: #{datepicker_options[:disabled].to_json},
+        altField: #{datepicker_options[:alt_field].to_json},
+        altFormat: #{datepicker_options[:alt_format].to_json},
+        appendText: #{datepicker_options[:append_text].to_json},
+        autoSize: #{datepicker_options[:auto_size].to_json},
+        buttonImage: #{datepicker_options[:button_image].to_json},
+        buttonImageOnly: #{datepicker_options[:button_image_only].to_json},
+        buttonText: #{datepicker_options[:button_text].to_json},
+        calculateWeek: #{datepicker_options[:calculate_week].to_json},
+        changeMonth: #{datepicker_options[:change_month].to_json},
+        changeYear: #{datepicker_options[:change_year].to_json},
+        closeText: #{datepicker_options[:close_text].to_json},
+        constrainInput: #{datepicker_options[:constrain_input].to_json},
+        currentText: #{datepicker_options[:current_text].to_json},
+        dateFormat: #{datepicker_options[:date_format].to_json},
+        dayNames: #{datepicker_options[:day_names].to_json},
+        dayNamesMin: #{datepicker_options[:day_names_min].to_json},
+        dayNamesShort: #{datepicker_options[:day_names_short].to_json},
+        defaultDate: #{datepicker_options[:default_date].to_json},
+        duration: #{datepicker_options[:duration].to_json},
+        firstDay: #{datepicker_options[:first_day].to_json},
+        gotoCurrent: #{datepicker_options[:goto_current].to_json},
+        hideIfNoPrevNext: #{datepicker_options[:hide_if_no_prev_next].to_json},
+        isRTL: #{datepicker_options[:is_rtl].to_json},
+        maxDate: #{datepicker_options[:max_date].to_json},
+        minDate: #{datepicker_options[:min_date].to_json},
+        monthNames: #{datepicker_options[:month_names].to_json},
+        monthNamesShort: #{datepicker_options[:month_names_short].to_json},
+        navigationAsDateFormat: #{datepicker_options[:navigation_as_date_format].to_json},
+        nextText: #{datepicker_options[:next_text].to_json},
+        numberOfMonths: #{datepicker_options[:number_of_months].to_json},
+        prevText: #{datepicker_options[:prev_text].to_json},
+        selectOtherMonths: #{datepicker_options[:select_other_months].to_json},
+        shortYearCutoff: #{datepicker_options[:short_year_cutoff].to_json},
+        showButtonPanel: #{datepicker_options[:show_button_panel].to_json},
+        showCurrentAtPos: #{datepicker_options[:show_current_at_pos].to_json},
+        showMonthAfterYear: #{datepicker_options[:show_month_after_year].to_json},
+        showOn: #{datepicker_options[:show_on].to_json},
+        showOptions: #{datepicker_options[:show_options].to_json},
+        showOtherMonths: #{datepicker_options[:show_other_months].to_json},
+        showWeek: #{datepicker_options[:show_week].to_json},
+        stepMonths: #{datepicker_options[:step_months].to_json},
+        weekHeader: #{datepicker_options[:week_header].to_json},
+        yearRange: #{datepicker_options[:year_range].to_json},
+        yearSuffix: #{datepicker_options[:year_suffix].to_json}
+      })
+    </script>
+    "
+    result
+  end
+
+  def date_range_field(from_name, to_name, from_value, to_value, format = '%Y-%m-%d', datepicker_options = {}, html_options = {})
+    from_id = html_options[:from_id] || 'datepicker-from-date'
+    to_id = html_options[:to_id] || 'datepicker-to-date'
+    return _('From') +' '+ date_field(from_name, from_value, format, datepicker_options, html_options.merge({:id => from_id})) +
+    ' ' + _('until') +' '+ date_field(to_name, to_value, format, datepicker_options, html_options.merge({:id => to_id}))
+  end
+
 protected
   def self.next_id_number
     if defined? @@id_num
@@ -2,4 +2,76 @@ class Box &lt; ActiveRecord::Base
   belongs_to :owner, :polymorphic => true
   acts_as_list :scope => 'owner_id = #{owner_id} and owner_type = \'#{owner_type}\''
   has_many :blocks, :dependent => :destroy, :order => 'position'
+
+  def acceptable_blocks
+    to_css_class_name central?  ? Box.acceptable_center_blocks : Box.acceptable_side_blocks
+  end
+
+  def central?
+    position == 1
+  end
+
+  def self.acceptable_center_blocks
+    [ ArticleBlock,
+      BlogArchivesBlock,
+      CategoriesBlock,
+      CommunitiesBlock,
+      EnterprisesBlock,
+      EnvironmentStatisticsBlock,
+      FansBlock,
+      FavoriteEnterprisesBlock,
+      FeedReaderBlock,
+      FriendsBlock,
+      HighlightsBlock,
+      LinkListBlock,
+      LoginBlock,
+      MainBlock,
+      MembersBlock,
+      MyNetworkBlock,
+      PeopleBlock,
+      ProfileImageBlock,
+      RawHTMLBlock,
+      RecentDocumentsBlock,
+      SellersSearchBlock,
+      TagsBlock ]
+  end
+
+  def self.acceptable_side_blocks
+    [ ArticleBlock,
+      BlogArchivesBlock,
+      CategoriesBlock,
+      CommunitiesBlock,
+      DisabledEnterpriseMessageBlock,
+      EnterprisesBlock,
+      EnvironmentStatisticsBlock,
+      FansBlock,
+      FavoriteEnterprisesBlock,
+      FeaturedProductsBlock,
+      FeedReaderBlock,
+      FriendsBlock,
+      HighlightsBlock,
+      LinkListBlock,
+      LocationBlock,
+      LoginBlock,
+      MembersBlock,
+      MyNetworkBlock,
+      PeopleBlock,
+      ProductsBlock,
+      ProfileImageBlock,
+      ProfileInfoBlock,
+      ProfileSearchBlock,
+      RawHTMLBlock,
+      RecentDocumentsBlock,
+      SellersSearchBlock,
+      SlideshowBlock,
+      TagsBlock
+    ]
+  end
+
+  private
+
+  def to_css_class_name(blocks)
+    blocks.map{ |block| block.to_s.underscore.tr('_', '-') }
+  end
+
 end
@@ -28,6 +28,8 @@ class Comment &lt; ActiveRecord::Base
   xss_terminate :only => [ :body, :title, :name ], :on => 'validation'
+  delegate :environment, :to => :source
+
   def action_tracker_target
     self.article.profile
   end
@@ -71,10 +71,7 @@ class Person &lt; Profile
     Friendship.find(:all, :conditions => { :friend_id => person.id}).each { |friendship| friendship.destroy }
   end
-  after_destroy :destroy_user
-  def destroy_user
-    self.user.destroy if self.user
-  end
+  belongs_to :user, :dependent => :delete
   def can_control_scrap?(scrap)
     begin
@@ -31,7 +31,7 @@ class Task &lt; ActiveRecord::Base
     end
   end
-  belongs_to :requestor, :class_name => 'Person', :foreign_key => :requestor_id
+  belongs_to :requestor, :class_name => 'Profile', :foreign_key => :requestor_id
   belongs_to :target, :foreign_key => :target_id, :polymorphic => true
   validates_uniqueness_of :code, :on => :create
@@ -30,7 +30,7 @@ class User &lt; ActiveRecord::Base
   after_create do |user|
     user.person ||= Person.new
-    user.person.attributes = user.person_data.merge(:identifier => user.login, :user_id => user.id, :environment_id => user.environment_id)
+    user.person.attributes = user.person_data.merge(:identifier => user.login, :user => user, :environment_id => user.environment_id)
     user.person.name ||= user.login
     user.person.visible = false unless user.activated?
     user.person.save!
@@ -88,13 +88,13 @@ class User &lt; ActiveRecord::Base
   attr_protected :activated_at
   # Virtual attribute for the unencrypted password
-  attr_accessor :password
+  attr_accessor :password, :name
   validates_presence_of     :login, :email
   validates_format_of       :login, :with => Profile::IDENTIFIER_FORMAT, :if => (lambda {|user| !user.login.blank?})
   validates_presence_of     :password,                   :if => :password_required?
-  validates_presence_of     :password_confirmation,      :if => :password_required?, :if => (lambda {|user| !user.password.blank?})
-  validates_length_of       :password, :within => 4..40, :if => :password_required?, :if => (lambda {|user| !user.password.blank?})
+  validates_presence_of     :password_confirmation,      :if => :password_required?
+  validates_length_of       :password, :within => 4..40, :if => :password_required?
   validates_confirmation_of :password,                   :if => :password_required?
   validates_length_of       :login,    :within => 2..40, :if => (lambda {|user| !user.login.blank?})
   validates_length_of       :email,    :within => 3..100, :if => (lambda {|user| !user.email.blank?})
@@ -228,7 +228,12 @@ class User &lt; ActiveRecord::Base
   end
   def name
-    person ? person.name : login
+    name = (self[:name] || login)
+    person.nil? ? name : (person.name || name)
+  end
+
+  def name= name
+   self[:name] = name
   end
   def enable_email!
@@ -274,6 +279,11 @@ class User &lt; ActiveRecord::Base
     15 # in minutes
   end
+
+  def not_require_password!
+    @is_password_required = false
+  end
+
   protected
     # before filter 
     def encrypt_password
@@ -282,9 +292,13 @@ class User &lt; ActiveRecord::Base
       self.password_type ||= User.system_encryption_method.to_s
       self.crypted_password = encrypt(password)
     end
-    
+
     def password_required?
-      crypted_password.blank? || !password.blank?
+      (crypted_password.blank? || !password.blank?) && is_password_required?
+    end
+
+    def is_password_required?
+      @is_password_required.nil? ? true : @is_password_required
     end
     def make_activation_code
@@ -292,6 +306,7 @@ class User &lt; ActiveRecord::Base
     end
     def deliver_activation_code
+      return if person.is_template?
       User::Mailer.deliver_activation_code(self) unless self.activation_code.blank?
     end
@@ -13,6 +13,8 @@
      <%= f.password_field :password %>
+     <%= @plugins.dispatch(:login_extra_contents).collect { |content| instance_eval(&content) }.join("") %>
+
      <% button_bar do %>
        <%= submit_button( 'login', _('Log in') )%>
        <% if is_thickbox %>
@@ -23,8 +25,13 @@
 <% end %>
 <% button_bar do %>
-  <%= button :add, _("New user"), :controller => 'account', :action => 'signup' %>
-  <%= button :help, _("I forgot my password!"), :controller => 'account', :action => 'forgot_password' %>
+  <% unless @plugins.dispatch(:allow_user_registration).include?(false) %>
+    <%= button :add, _("New user"), :controller => 'account', :action => 'signup' %>
+  <% end %>
+
+  <% unless @plugins.dispatch(:allow_password_recovery).include?(false) %>
+    <%= button :help, _("I forgot my password!"), :controller => 'account', :action => 'forgot_password' %>
+  <% end %>
 <% end %>
 </div><!-- end class="login-box" -->
@@ -9,25 +9,30 @@
   @user ||= User.new
 %>
-    <% labelled_form_for :user, @user,
-       :url => login_url do |f| %>
+    <% labelled_form_for :user, @user, :url => login_url do |f| %>
-         <%= f.text_field :login, :onchange => 'this.value = convToValidLogin( this.value )' %>
+      <%= f.text_field :login, :onchange => 'this.value = convToValidLogin( this.value )' %>
-         <%= f.password_field :password %>
+      <%= f.password_field :password %>
+
+      <%= @plugins.dispatch(:login_extra_contents).collect { |content| instance_eval(&content) }.join("") %>
       <% button_bar do %>
         <%= submit_button( 'login', _('Log in') )%>
-        <%= link_to content_tag( 'span', _('New user') ),
-            { :controller => 'account', :action => 'signup' },
-            :class => 'button with-text icon-add' %>
+        <% unless @plugins.dispatch(:allow_user_registration).include?(false) %>
+          <%= link_to content_tag( 'span', _('New user') ),
+              { :controller => 'account', :action => 'signup' },
+              :class => 'button with-text icon-add' %>
+        <% end %>
       <% end %>
     <% end %>
-    <p class="forgot-passwd">
-      <%= link_to _("I forgot my password!"), :controller => 'account', :action => 'forgot_password' %>
-    </p>
+    <% unless @plugins.dispatch(:allow_password_recovery).include?(false) %>
+      <p class="forgot-passwd">
+        <%= link_to _("I forgot my password!"), :controller => 'account', :action => 'forgot_password' %>
+      </p>
+    <% end %>
     </div>
@@ -0,0 +1,10 @@
+<% block_types.in_groups_of(2) do |block1, block2| %>
+  <div style='float: left; width: 48%; padding-top: 2px;'>
+    <%= labelled_radio_button(block1.description, :type, block1.name) %>
+  </div>
+  <% if block2 %>
+    <div style='float: left; width: 48%; padding-top: 2px;'>
+      <%= labelled_radio_button(block2.description, :type, block2.name) %>
+    </div>
+  <% end %>
+<% end %>
 <strong><%= _('Highlights') %></strong>
-<div id='edit-highlights-block'>
+<div id='edit-highlights-block' style='width:450px'>
 <table id='highlights' class='noborder'>
   <tr><th><%= _('Image') %></th><th><%= _('Address') %></th><th><%= _('Position') %></th><th><%= _('Title') %></th></tr>
   <% for image in @block.images do %>
 <strong><%= _('Links') %></strong>
-<div id='edit-link-list-block'>
+<div id='edit-link-list-block' style='width:450px'>
 <table id='links' class='noborder'>
   <tr><th><%= _('Icon') %></th><th><%= _('Name') %></th><th><%= _('Address') %></th></tr>
   <% for link in @block.links do %>
-<%= labelled_form_field(_('HTML code'), text_area(:block, :html, :style => 'width: 100%', :rows => 15)) %>
+<%= labelled_form_field(_('HTML code'), text_area(:block, :html, :style => 'width: 90%', :rows => 15)) %>
-<% form_tag do %>
-
-  <p><%= _('In what area do you want to put your new block?') %></p>
-
-  <%# FIXME hardcoded stuff %>
-  <%= select_tag('box_id', options_for_select(@boxes.select { |item| item.position != 1 }.map {|item| [ _("Area %d") % item.position, item.id]})) %>
-  
-  <p><%= _('Select the type of block you want to add to your page.') %></p>
-  
-  <% @block_types.in_groups_of(2) do |block1, block2| %>
-    <div style='float: left; width: 48%; padding-top: 2px;'>
-      <%= radio_button_tag('type', block1.name) %>
-      <%= label_tag "type_#{block1.name.downcase}", block1.description %>
+<div style='height:350px'>
+  <% form_tag do %>
+
+    <p><%= _('In what area do you want to put your new block?') %></p>
+
+    <% @boxes.each do |box| %>
+      <%= labelled_radio_button(_("Area %d") % box.position, :box_id, box.id, box.central?, { :class => 'box-position', 'data-position' => box.position })  %>
+    <% end %>
+
+    <script type="text/javascript">
+      (function ($) {
+        $(document).ready(function () {
+          $(".box-position").live('change', function () {
+            if ($(this).attr('data-position') == '1') {
+              $('#center-block-types').show();
+              $('#side-block-types').hide();
+            } else {
+              $('#center-block-types').hide();
+              $('#side-block-types').show();
+            };
+          });
+      })})(jQuery);
+    </script>
+
+    <p><%= _('Select the type of block you want to add to your page.') %></p>
+
+    <div id='center-block-types'>
+      <%= render :partial => 'block_types', :locals => { :block_types => @center_block_types } %>
     </div>
-    <% if block2 %>
-      <div style='float: left; width: 48%; padding-top: 2px;'>
-        <%= radio_button_tag('type', block2.name) %>
-        <%= label_tag "type_#{block2.name.downcase}", block2.description %>
-      </div>
+
+    <div id='side-block-types' style='display:none'>
+      <%= render :partial => 'block_types', :locals => { :block_types => @side_block_types } %>
+    </div>
+
+    <br style='clear: both'/>
+
+    <% button_bar do %>
+      <%= submit_button(:add, _("Add")) %>
+      <%= colorbox_close_button(_('Close')) %>
     <% end %>
-  <% end %>
-  <br style='clear: both'/>
-  
-  <% button_bar do %>
-    <%= submit_button(:add, _("Add")) %>
-    <%= lightbox_close_button(_('Close')) %>
-  <% end %>
-<% end %>
+  <% end %>
+</div>
-<h2><%= _('Editing block') %></h2>
+<div style='width: 500px;'>
+  <h2><%= _('Editing block') %></h2>
-<% form_tag(:action => 'save', :id => @block.id) do %>
+  <% form_tag(:action => 'save', :id => @block.id) do %>
-  <%= labelled_form_field(_('Custom title for this block: '), text_field(:block, :title, :maxlength => 20)) %>
+    <%= labelled_form_field(_('Custom title for this block: '), text_field(:block, :title, :maxlength => 20)) %>
-  <%= render :partial => partial_for_class(@block.class) %>
+    <%= render :partial => partial_for_class(@block.class) %>
-  <%= labelled_form_field _('Display this block:'), '' %>
-  <div style='margin-left: 10px'>
-    <%= radio_button(:block, :display, 'always') %>
-    <%= label_tag('block_display_always', _('In all pages')) %>
-    <br/>
-    <%= radio_button(:block, :display, 'home_page_only') %>
-    <%= label_tag('block_display_home_page_only', _('Only in the homepage')) %>
-    <br/>
-    <%= radio_button(:block, :display, 'except_home_page') %>
-    <%= label_tag('block_display_except_home_page', _('In all pages, except in the homepage')) %>
-    <br/>
-    <%= radio_button(:block, :display, 'never') %>
-    <%= label_tag('block_display_never', _("Don't display")) %>
-  </div>
+    <%= labelled_form_field _('Display this block:'), '' %>
+    <div style='margin-left: 10px'>
+      <%= radio_button(:block, :display, 'always') %>
+      <%= label_tag('block_display_always', _('In all pages')) %>
+      <br/>
+      <%= radio_button(:block, :display, 'home_page_only') %>
+      <%= label_tag('block_display_home_page_only', _('Only in the homepage')) %>
+      <br/>
+      <%= radio_button(:block, :display, 'except_home_page') %>
+      <%= label_tag('block_display_except_home_page', _('In all pages, except in the homepage')) %>
+      <br/>
+      <%= radio_button(:block, :display, 'never') %>
+      <%= label_tag('block_display_never', _("Don't display")) %>
+    </div>
-  <%= labelled_form_field(_('Show for:'), select(:block, :language, [ [ _('all languages'), 'all']] + Noosfero.locales.map {|key, value| [value, key]} )) %>
+    <%= labelled_form_field(_('Show for:'), select(:block, :language, [ [ _('all languages'), 'all']] + Noosfero.locales.map {|key, value| [value, key]} )) %>
-  <% button_bar do %>
-    <%= submit_button(:save, _('Save')) %>
-    <%= lightbox_close_button(_('Cancel')) %>
-  <% end %>
+    <% button_bar do %>
+      <%= submit_button(:save, _('Save')) %>
+      <%= colorbox_close_button(_('Cancel')) %>
+    <% end %>
-<% end %>
+  <% end %>
+</div>
 <h1><%= _('Editing sideboxes')%></h1>
 <% button_bar do %>
-  <%= lightbox_button('add', _('Add a block'), { :action => 'add_block' }) %> 
+  <%= colorbox_button('add', _('Add a block'), { :action => 'add_block' }) %>
   <%= button(:back, _('Back to control panel'), :controller => (profile.nil? ? 'admin_panel': 'profile_editor')) %>
 <% end %>
@@ -49,13 +49,13 @@
         <%= article.class.short_description %>
       </td>
       <td class="article-controls">
-        <%= button_without_text :edit, _('Edit'), :action => 'edit', :id => article.id %>
+        <%= expirable_button article, :edit, _('Edit'), {:action => 'edit', :id => article.id} if !remove_content_button(:edit) %>
         <%= button_without_text :eyes, _('Public view'), article.view_url %>
-        <%= display_spread_button(profile, article) unless article.folder? %>
-        <% if !environment.enabled?('cant_change_homepage') %>
-          <%= button_without_text :home, _('Use as homepage'), { :action => 'set_home_page', :id => article.id }, :method => :post %>
+        <%= display_spread_button(profile, article) unless article.folder? || remove_content_button(:spread)%>
+        <% if !environment.enabled?('cant_change_homepage') && !remove_content_button(:home) %>
+          <%= expirable_button article, :home, _('Use as homepage'), { :action => 'set_home_page', :id => article.id }, :method => :post %>
         <% end %>
-        <%= display_delete_button(article) %>
+        <%= display_delete_button(article) if !remove_content_button(:delete) %>
       </td>
     </tr>
   <% end %>
 <div<%= user && " class='logged-in'" %>>
   <div id="article-actions">
-    <% if @page.allow_edit?(user) %>
-      <%= link_to content_tag( 'span', label_for_edit_article(@page) ),
-          profile.admin_url.merge({ :controller => 'cms', :action => 'edit', :id => @page.id }),
-          :class => 'button with-text icon-edit' %>
+
+    <% if @page.allow_edit?(user) && !remove_content_button(:edit) %>
+      <% content = content_tag('span', label_for_edit_article(@page)) %>
+      <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'edit', :id => @page.id }) %>
+      <%= expirable_button @page, :edit, content, url %>
     <% end %>
-    <% if @page != profile.home_page && !@page.has_posts?  && @page.allow_delete?(user) %>
-      <%= link_to content_tag( 'span', _('Delete') ),
-        profile.admin_url.merge({ :controller => 'cms', :action => 'destroy', :id => @page}),
-        :method => :post,
-        :class => 'button with-text icon-delete',
-        :confirm => delete_article_message(@page) %>
+    <% if @page != profile.home_page && !@page.has_posts?  && @page.allow_delete?(user) && !remove_content_button(:delete)%>
+      <% content = content_tag( 'span', _('Delete') ) %>
+      <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'destroy', :id => @page}) %>
+      <% options = {:method => :post, :confirm => delete_article_message(@page)} %>
+      <%= expirable_button @page, :delete, content, url, options %>
     <% end %>
-    <% if !@page.folder? && @page.allow_spread?(user) %>
+    <% if !@page.folder? && @page.allow_spread?(user) && !remove_content_button(:spread) %>
+      <% content = content_tag( 'span', _('Spread this') ) %>
+      <% url = nil %>
       <% if profile.kind_of?(Person) %>
-        <%= link_to content_tag( 'span', _('Spread this') ),
-            profile.admin_url.merge({ :controller => 'cms', :action => 'publish', :id => @page }),
-            :class => 'button with-text icon-spread' %>
+        <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish', :id => @page }) %>
       <% elsif profile.kind_of?(Community) && environment.portal_community %>
-        <%= link_to content_tag( 'span', _('Spread this') ),
-            profile.admin_url.merge({ :controller => 'cms', :action => 'publish_on_portal_community', :id => @page }),
-            :class => 'button with-text icon-spread' %>
+        <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish_on_portal_community', :id => @page }) %>
       <% end %>
+      <%= expirable_button @page, :spread, content, url if url %>
     <% end %>
     <% if !@page.gallery? && @page.allow_create?(user) %>
-      <%= link_to _('Add translation'),
-          profile.admin_url.merge(:controller => 'cms', :action => 'new',
-                                  :parent_id => (@page.folder? ? @page : (@page.parent.nil? ? nil : @page.parent)),
-                                  :type => @page.type, :article => { :translation_of_id => @page.native_translation.id }),
-          :class => 'button with-text icon-locale' if @page.translatable? && !@page.native_translation.language.blank? %>
+      <% if @page.translatable? && !@page.native_translation.language.blank? && !remove_content_button(:locale) %>
+        <% content = _('Add translation') %>
+        <% parent_id = (@page.folder? ? @page : (@page.parent.nil? ? nil : @page.parent)) %>
+        <% url = profile.admin_url.merge(:controller => 'cms', :action => 'new', :parent_id => parent_id, :type => @page.type, :article => { :translation_of_id => @page.native_translation.id })%>
+        <%= expirable_button @page, :locale, content, url %>
+      <% end %>
+
       <%= colorbox_button(:new, label_for_new_article(@page), profile.admin_url.merge(:controller => 'cms', :action => 'new', :parent_id => (@page.folder? ? @page : (@page.parent.nil? ? nil : @page.parent)))) %>
     <% end %>
@@ -40,8 +41,11 @@
       <%= button('upload-file', _('Upload files'), profile.admin_url.merge(:controller => 'cms', :action => 'upload_files', :parent_id => (@page.folder? ? @page : @page.parent))) %>
     <% end %>
-    <% if !@page.allow_create?(user) && profile.community? && (@page.blog? || @page.parent && @page.parent.blog?) %>
-      <%= link_to content_tag( 'span', _('Suggest an article') ), profile.admin_url.merge({ :controller => 'cms', :action => 'suggest_an_article'}), :id => 'suggest-article-link', :class => 'button with-text icon-new' %>
+    <% if !@page.allow_create?(user) && profile.community? && (@page.blog? || @page.parent && @page.parent.blog?) && !remove_content_button(:suggest) %>
+      <% content = content_tag( 'span', _('Suggest an article') ) %>
+      <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'suggest_an_article'}) %>
+      <% options = {:id => 'suggest-article-link'} %>
+      <%= expirable_button @page, :suggest, content, url, options %>
     <% end %>
     <%= report_abuse(profile, :link, @page) %>
@@ -63,6 +63,11 @@
         <% end %>
       <% end %>
+      <% if comment.author && comment.author == user %>
+        &nbsp;
+        <%= expirable_comment_link comment, :edit, _('Edit'), {:action => 'edit_comment', :id => comment.id, :profile => profile.identifier} %>
+      <% end %>
+
       <% if logged_in? && (user == profile || user == comment.author || user.has_permission?(:moderate_comments, profile)) %>
         &nbsp;
         <%= link_to_function(_('Remove'), 'remove_comment(this, %s, %s); return false ;' % [url_for(:profile => params[:profile], :remove_comment => comment.id, :view => params[:view]).to_json, _('Are you sure you want to remove this comment and all its replies?').to_json], :class => 'comment-footer comment-footer-link comment-footer-hide remove-children') %>
@@ -32,15 +32,17 @@ function submit_comment_form(button) {
 <div class="post_comment_box <%= @form_div %>">
-<h4 onclick="var d = jQuery(this).parent('.post_comment_box');
-             if (d.hasClass('closed')) {
-               d.removeClass('closed');
-               d.addClass('opened');
-               d.find('input[name=comment[title]], textarea').val('');
-               d.find('.comment_form input[name=comment[<%= focus_on %>]]').focus();
-             }">
-  <%= content_tag('a', '', :name => 'comment_form') + _('Post a comment') %>
-</h4>
+<% if display_link %>
+  <h4 onclick="var d = jQuery(this).parent('.post_comment_box');
+               if (d.hasClass('closed')) {
+                 d.removeClass('closed');
+                 d.addClass('opened');
+                 d.find('input[name=comment[title]], textarea').val('');
+                 d.find('.comment_form input[name=comment[<%= focus_on %>]]').focus();
+               }">
+    <%= content_tag('a', '', :name => 'comment_form') + _('Post a comment') %>
+  </h4>
+<% end %>
 <% unless pass_without_comment_captcha? %>
   <div id="recaptcha-container" style="display: none">
@@ -59,7 +61,7 @@ function submit_comment_form(button) {
   </script>
 <% end %>
-<% form_tag( url_for(@page.view_url.merge({:only_path => true})), { :class => 'comment_form' } ) do %>
+<% form_tag( url, { :class => 'comment_form' } ) do %>
   <%= hidden_field_tag(:confirm, 'false') %>
   <%= required_fields_message %>
@@ -84,7 +86,11 @@ function submit_comment_form(button) {
   <% button_bar do %>
     <%= submit_button('add', _('Post comment'), :onclick => "submit_comment_form(this); return false") %>
-    <%= button_to_function :cancel, _('Cancel'), "f=jQuery(this).parents('.post_comment_box'); f.removeClass('opened'); f.addClass('closed'); return false" %>
+    <% if cancel_triggers_hide %>
+      <%= button_to_function :cancel, _('Cancel'), "f=jQuery(this).parents('.post_comment_box'); f.removeClass('opened'); f.addClass('closed'); return false" %>
+    <% else %>
+      <%= button('cancel', _('Cancel'), {:action => 'view_page', :profile => profile.identifier, :page => @comment.article.explode_path})%>
+    <% end %>
   <% end %>
 <% end %>
@@ -0,0 +1,3 @@
+<h1><%= _('Edit comment') %></h1>
+
+<%= render :partial => 'comment_form', :locals => {:url => {:action => 'edit_comment', :profile => profile.identifier}, :display_link => false, :cancel_triggers_hide => false} %>
@@ -98,7 +98,7 @@
   </ul>
   <% if @page.accept_comments? %>
-    <div id="page-comment-form"><%= render :partial => 'comment_form' %></div>
+    <div id="page-comment-form"><%= render :partial => 'comment_form', :locals => {:url => url_for(@page.view_url.merge({:only_path => true})), :display_link => true, :cancel_triggers_hide => true}%></div>
   <% end %>
 </div><!-- end class="comments" -->
@@ -2,7 +2,8 @@
 'jquery.noconflict.js', 'jquery.cycle.all.min.js', 'thickbox.js', 'lightbox', 'colorbox',
 'jquery-ui-1.8.2.custom.min', 'jquery.scrollTo', 'jquery.form.js', 'jquery-validation/jquery.validate',
 'jquery.cookie', 'jquery.ba-bbq.min.js', 'reflection', 'jquery.tokeninput',
-'add-and-join', 'report-abuse', 'catalog', 'manage-products', :cache => 'cache-general' %>
+'add-and-join', 'report-abuse', 'catalog', 'manage-products',
+'jquery-ui-timepicker-addon', :cache => 'cache-general' %>
 <% language = FastGettext.locale %>
 <% %w{messages methods}.each do |type| %>
@@ -56,10 +56,18 @@
             <%= usermenu_logged_in %>
           </span>
           <span class='not-logged-in' style='display: none'>
-              <%= _("<span class='login'>%s</span> <span class='or'>or</span> <span class='signup'>%s</span>") % [thickbox_inline_popup_link('<i class="icon-menu-login"></i><strong>' + _('Login') + '</strong>', login_url, 'inlineLoginBox', :id => 'link_login'), link_to('<strong>' + _('Sign up') + '</strong>', :controller => 'account', :action => 'signup') ] %>
+
+              <%= _("<span class='login'>%s</span>") % thickbox_inline_popup_link('<i class="icon-menu-login"></i><strong>' + _('Login') + '</strong>', login_url, 'inlineLoginBox', :id => 'link_login') %>
+              <%= @plugins.dispatch(:alternative_authentication_link).collect { |content| instance_eval(&content) }.join("") %>
+
               <div id='inlineLoginBox' style='display: none;'>
                 <%= render :file => 'account/login', :locals => { :is_thickbox => true } %>
               </div>
+
+              <% unless @plugins.dispatch(:allow_user_registration).include?(false) %>
+              <%= _("<span class='or'>or</span> <span class='signup'>%s</span>") % link_to('<strong>' + _('Sign up') + '</strong>', :controller => 'account', :action => 'signup')%>
+              <% end %>
+
           </span>
           <form action="/search" class="search_form" method="get" class="clean">
             <input name="query" size="15" title="<%=_('Search...')%>" onfocus="this.form.className='focused';" onblur="this.form.className=''" />
@@ -62,6 +62,10 @@
       </script>
     <% end %>
     <%= report_abuse(comment.author, :comment_link, comment) if comment.author %>
+    <% if comment.author && comment.author == user %>
+      <%= expirable_comment_link comment, :edit, _('Edit'), {:action => 'edit_comment', :id => comment.id, :profile => profile.identifier} %>
+      <%= content_tag('span', ' | ', :class => 'comment-footer comment-footer-hide') %>
+    <% end %>
     <%= link_to_function _('Reply'),
         "var f = add_comment_reply_form(this, %s); f.find('input[name=comment[title]], textarea').val(''); return false" % comment.id,
         :class => 'comment-footer comment-footer-link comment-footer-hide',
@@ -50,13 +50,13 @@
   <% fields_for "tasks[#{task.id}][task]", task do |f| %>
     <% if task.accept_details %>
       <div id="on-accept-information-<%=task.id%>" style="display: none">
-        <%= render :partial => partial_for_task_class(task.class, :accept_details), :locals => {:task => task, :f => f} %>
+        <%= render :partial => partial_for_class(task.class, :accept_details), :locals => {:task => task, :f => f} %>
       </div>
     <% end %>
     <% if task.reject_details %>
       <div id="on-reject-information-<%=task.id%>" style="display: none">
-        <%= render :partial => partial_for_task_class(task.class, :reject_details), :locals => {:task => task, :f => f} %>
+        <%= render :partial => partial_for_class(task.class, :reject_details), :locals => {:task => task, :f => f} %>
       </div>
     <% end %>
   <% end %>
@@ -19,6 +19,7 @@ ActionController::Routing::Routes.draw do |map|
   # -- just remember to delete public/index.html.
   # You can have the root of your site routed by hooking up ''
+  map.root :controller => "home", :conditions => { :if => lambda { |env| !Domain.hosting_profile_at(env[:host]) } }
   map.connect '', :controller => "home", :conditions => { :if => lambda { |env| !Domain.hosting_profile_at(env[:host]) } }
   map.home 'site/:action', :controller => 'home'
@@ -121,9 +122,12 @@ ActionController::Routing::Routes.draw do |map|
   # cache stuff - hack
   map.cache 'public/:action/:id', :controller => 'public'
+  map.connect ':profile/edit_comment/:id/*page', :controller => 'content_viewer', :action => 'edit_comment', :profile => /#{Noosfero.identifier_format}/
+
   # match requests for profiles that don't have a custom domain
   map.homepage ':profile/*page', :controller => 'content_viewer', :action => 'view_page', :profile => /#{Noosfero.identifier_format}/, :conditions => { :if => lambda { |env| !Domain.hosting_profile_at(env[:host]) } }
+
   # match requests for content in domains hosted for profiles
   map.connect '*page', :controller => 'content_viewer', :action => 'view_page'
@@ -2,4 +2,7 @@
 set -e
-cd /usr/share/noosfero && su noosfero -c "./script/console production"
+environment="$1"
+test -z "$environment" && environment=production
+
+cd /usr/share/noosfero && su noosfero -c "./script/console $environment"
@@ -12,7 +12,11 @@ class Noosfero::Plugin
     end
     def init_system
-      Dir.glob(File.join(Rails.root, 'config', 'plugins', '*')).select do |entry|
+      enabled_plugins = Dir.glob(File.join(Rails.root, 'config', 'plugins', '*'))
+      if Rails.env.test? && !enabled_plugins.include?(File.join(Rails.root, 'config', 'plugins', 'foo'))
+        enabled_plugins << File.join(Rails.root, 'plugins', 'foo')
+      end
+      enabled_plugins.select do |entry|
         File.directory?(entry)
       end.each do |dir|
         plugin_name = File.basename(dir)
@@ -31,6 +35,11 @@ class Noosfero::Plugin
         if plugin_dependencies_ok
           Rails.configuration.controller_paths << File.join(dir, 'controllers')
           ActiveSupport::Dependencies.load_paths << File.join(dir, 'controllers')
+          controllers_folders = %w[public profile myprofile admin]
+          controllers_folders.each do |folder|
+            Rails.configuration.controller_paths << File.join(dir, 'controllers', folder)
+            ActiveSupport::Dependencies.load_paths << File.join(dir, 'controllers', folder)
+          end
           [ ActiveSupport::Dependencies.load_paths, $:].each do |path|
             path << File.join(dir, 'models')
             path << File.join(dir, 'lib')
@@ -211,28 +220,6 @@ class Noosfero::Plugin
     nil
   end
-  # This is a generic hotspot for all controllers on Noosfero.
-  # If any plugin wants to define filters to run on any controller, the name of
-  # the hotspot must be in the following form: <underscored_controller_name>_filters.
-  # Example: for ProfileController the hotspot is profile_controller_filters
-  #
-  # -> Adds a filter to a controller
-  # returns = { :type => type,
-  #             :method_name => method_name,
-  #             :options => {:opt1 => opt1, :opt2 => opt2},
-  #             :block => Proc or lambda block}
-  #   type = 'before_filter' or 'after_filter'
-  #   method_name = The name of the filter
-  #   option = Filter options, like :only or :except
-  #   block = Block that the filter will call
-  def method_missing(method, *args, &block)
-    if method.to_s =~ /^(.+)_controller_filters$/
-      []
-    else
-      super
-    end
-  end
-
   # This method will be called just before a comment is saved to the database.
   #
   # It can modify the comment in several ways. In special, a plugin can call
@@ -333,4 +320,71 @@ class Noosfero::Plugin
     nil
   end
+  # -> Add an alternative authentication method.
+  # Your plugin have to make the access control and return the logged user.
+  # returns = User
+  def alternative_authentication
+    nil
+  end
+
+  # -> Adds adicional link to make the user authentication
+  # returns = lambda block that creates html code
+  def alternative_authentication_link
+    nil
+  end
+
+  # -> Allow or not user registration
+  # returns = boolean
+  def allow_user_registration
+    true
+  end
+
+  # -> Allow or not password recovery by users
+  # returns = boolean
+  def allow_password_recovery
+    true
+  end
+
+  # -> Adds fields to the login form
+  # returns = lambda block that creates html code
+  def login_extra_contents
+    nil
+  end
+
+  def method_missing(method, *args, &block)
+    # This is a generic hotspot for all controllers on Noosfero.
+    # If any plugin wants to define filters to run on any controller, the name of
+    # the hotspot must be in the following form: <underscored_controller_name>_filters.
+    # Example: for ProfileController the hotspot is profile_controller_filters
+    #
+    # -> Adds a filter to a controller
+    # returns = { :type => type,
+    #             :method_name => method_name,
+    #             :options => {:opt1 => opt1, :opt2 => opt2},
+    #             :block => Proc or lambda block}
+    #   type = 'before_filter' or 'after_filter'
+    #   method_name = The name of the filter
+    #   option = Filter options, like :only or :except
+    #   block = Block that the filter will call
+    if method.to_s =~ /^(.+)_controller_filters$/
+      []
+    # -> Removes the action button from the content
+    # returns = boolean
+    elsif method.to_s =~ /^content_remove_(#{content_actions.join('|')})$/
+      nil
+    # -> Expire the action button from the content
+    # returns = string with reason of expiration
+    elsif method.to_s =~ /^content_expire_(#{content_actions.join('|')})$/
+      nil
+    else
+      super
+    end
+  end
+
+  private
+
+  def content_actions
+    %w[edit delete spread locale suggest home]
+  end
+
 end
-Dir.glob(File.join(Rails.root, 'config', 'plugins', '*', 'controllers')) do |dir|
-  plugin_name = File.basename(File.dirname(dir))
+plugins_root = Rails.env.test? ? 'plugins' : File.join('config', 'plugins')
+
+Dir.glob(File.join(Rails.root, plugins_root, '*', 'controllers')) do |controllers_dir|
+  prefixes_by_folder = {'public' => 'plugin',
+                        'profile' => 'profile/:profile/plugin',
+                        'myprofile' => 'myprofile/:profile/plugin',
+                        'admin' => 'admin/plugin'}
+
+  controllers_by_folder = prefixes_by_folder.keys.inject({}) do |hash, folder|
+    hash.merge!({folder => Dir.glob(File.join(controllers_dir, folder, '*')).map {|full_names| File.basename(full_names).gsub(/_controller.rb$/,'')}})
+  end
+
+  plugin_name = File.basename(File.dirname(controllers_dir))
+
+  controllers_by_folder.each do |folder, controllers|
+    controllers.each do |controller|
+      controller_name = controller.gsub("#{plugin_name}_plugin_",'')
+      map.connect "#{prefixes_by_folder[folder]}/#{plugin_name}/#{controller_name}/:action/:id", :controller => controller
+    end
+  end
+
   map.connect 'plugin/' + plugin_name + '/:action/:id', :controller => plugin_name + '_plugin'
-  map.connect 'profile/:profile/plugins/' + plugin_name + '/:action/:id', :controller => plugin_name + '_plugin_profile'
+  map.connect 'profile/:profile/plugin/' + plugin_name + '/:action/:id', :controller => plugin_name + '_plugin_profile'
   map.connect 'myprofile/:profile/plugin/' + plugin_name + '/:action/:id', :controller => plugin_name + '_plugin_myprofile'
   map.connect 'admin/plugin/' + plugin_name + '/:action/:id', :controller => plugin_name + '_plugin_admin'
 end
-
@@ -0,0 +1,147 @@
+class CustomFormsPluginMyprofileController < MyProfileController
+
+  protect 'post_content', :profile
+  def index
+    @forms = CustomFormsPlugin::Form.from(profile)
+  end
+
+  def create
+    @form = CustomFormsPlugin::Form.new(:profile => profile)
+    @fields = []
+    @empty_field = CustomFormsPlugin::Field.new
+    if request.post?
+      begin
+        @form.update_attributes!(params[:form])
+        params[:fields] = format_kind(params[:fields])
+        params[:fields] = format_choices(params[:fields])
+        params[:fields] = set_form_id(params[:fields], @form.id)
+        create_fields(new_fields(params))
+        session['notice'] = _('Form created')
+        redirect_to :action => 'index'
+      rescue Exception => exception
+        logger.error(exception.to_s)
+        session['notice'] = _('Form could not be created')
+      end
+    end
+  end
+
+  def edit
+    @form = CustomFormsPlugin::Form.find(params[:id])
+    @fields = @form.fields
+    @empty_field = CustomFormsPlugin::TextField.new
+    if request.post?
+      begin
+        @form.update_attributes!(params[:form])
+        params[:fields] = format_kind(params[:fields])
+        params[:fields] = format_choices(params[:fields])
+        remove_fields(params, @form)
+        create_fields(new_fields(params))
+        update_fields(edited_fields(params))
+        session['notice'] = _('Form updated')
+        redirect_to :action => 'index'
+      rescue Exception => exception
+        logger.error(exception.to_s)
+        session['notice'] = _('Form could not be updated')
+      end
+    end
+  end
+
+  def remove
+    @form = CustomFormsPlugin::Form.find(params[:id])
+    begin
+      @form.destroy
+      session[:notice] = _('Form removed')
+    rescue
+      session[:notice] = _('Form could not be removed')
+    end
+    redirect_to :action => 'index'
+  end
+
+  def submissions
+    @form = CustomFormsPlugin::Form.find(params[:id])
+    @submissions = @form.submissions
+  end
+
+  def show_submission
+    @submission = CustomFormsPlugin::Submission.find(params[:id])
+    @form = @submission.form
+  end
+
+  private
+
+  def new_fields(params)
+    result = params[:fields].map {|id, hash| hash.has_key?(:real_id) ? nil : hash}.compact
+    result.delete_if {|field| field[:name].blank?}
+    result
+  end
+
+  def edited_fields(params)
+    params[:fields].map {|id, hash| hash.has_key?(:real_id) ? hash : nil}.compact
+  end
+
+  def create_fields(fields)
+    fields.each do |field|
+      case field[:type]
+      when 'text_field'
+        CustomFormsPlugin::TextField.create!(field)
+      when 'select_field'
+        CustomFormsPlugin::SelectField.create!(field)
+      else
+        CustomFormsPlugin::Field.create!(field)
+      end
+    end
+  end
+
+  def update_fields(fields)
+    fields.each do |field_attrs|
+      field = CustomFormsPlugin::Field.find(field_attrs.delete(:real_id))
+      field.attributes = field_attrs
+      field.save! if field.changed?
+    end
+  end
+
+  def format_kind(fields)
+    fields.each do |id, field|
+      next if field[:kind].blank?
+      kind = field.delete(:kind)
+      case kind
+      when 'radio'
+        field[:list] = false
+        field[:multiple] = false
+      when 'check_box'
+        field[:list] = false
+        field[:multiple] = true
+      when 'select'
+        field[:list] = true
+        field[:multiple] = false
+      when 'multiple_select'
+        field[:list] = true
+        field[:multiple] = true
+      end
+    end
+    fields
+  end
+
+  def format_choices(fields)
+    fields.each do |id, field|
+      next if !field.has_key?(:choices)
+      field[:choices] = field[:choices].map {|key, value| value}.inject({}) do |result, choice| 
+        hash = (choice[:name].blank? || choice[:value].blank?) ? {} : {choice[:name] => choice[:value]}
+        result.merge!(hash)
+      end
+    end
+    fields
+  end
+
+  def remove_fields(params, form)
+    present_fields = params[:fields].map{|id, value| value}.collect {|field| field[:real_id]}.compact
+    form.fields.each {|field| field.destroy if !present_fields.include?(field.id.to_s) }
+  end
+
+  def set_form_id(fields, form_id)
+    fields.each do |id, field|
+      field[:form_id] = form_id
+    end
+    fields
+  end
+end
@@ -0,0 +1,45 @@
+class CustomFormsPluginProfileController < ProfileController
+
+  before_filter :has_access, :show
+
+  def show
+    @form = CustomFormsPlugin::Form.find(params[:id])
+    if user
+      @submission ||= CustomFormsPlugin::Submission.find_by_form_id_and_profile_id(@form.id,user.id)
+      @submission ||= CustomFormsPlugin::Submission.new(:form_id => @form.id, :profile_id => user.id)
+    else
+      @submission ||= CustomFormsPlugin::Submission.new(:form_id => @form.id)
+    end
+    if request.post?
+      begin
+        extend(CustomFormsPlugin::Helper)
+        answers = build_answers(params[:submission], @form)
+        failed_answers = answers.select {|answer| !answer.valid? }
+        if failed_answers.empty?
+          if !user
+            @submission.author_name = params[:author_name]
+            @submission.author_email = params[:author_email]
+          end
+          @submission.save!
+          answers.map {|answer| answer.submission = @submission; answer.save!}
+        else
+          @submission.valid?
+          failed_answers.each do |answer|
+            @submission.errors.add(answer.field.name.to_sym, answer.errors[answer.field.slug.to_sym])
+          end
+        end
+        session[:notice] = _('Submission saved')
+        redirect_to :action => 'show'
+      rescue
+        session[:notice] = _('Submission could not be saved')
+      end
+    end
+  end
+
+  private
+
+  def has_access
+    form = CustomFormsPlugin::Form.find(params[:id])
+    render_access_denied if !form.accessible_to(user)
+  end
+end
@@ -0,0 +1,20 @@
+class CreateCustomFormsPluginForms < ActiveRecord::Migration
+  def self.up
+    create_table :custom_forms_plugin_forms do |t|
+      t.string :name
+      t.string :slug
+      t.text :description
+      t.references :profile
+      t.datetime :begining
+      t.datetime :ending
+      t.boolean :report_submissions, :default => false
+      t.boolean :on_membership, :default => false
+      t.string :access
+      t.timestamps
+    end 
+  end
+
+  def self.down
+    drop_table :custom_forms_plugin_forms
+  end
+end
@@ -0,0 +1,21 @@
+class CreateCustomFormsPluginFields < ActiveRecord::Migration
+  def self.up
+    create_table :custom_forms_plugin_fields do |t|
+      t.string :name
+      t.string :slug
+      t.string :type
+      t.string :default_value
+      t.string :choices
+      t.float  :minimum
+      t.float  :maximum
+      t.references :form
+      t.boolean :mandatory, :default => false
+      t.boolean :multiple
+      t.boolean :list
+    end
+  end
+
+  def self.down
+    drop_table :custom_forms_plugin_fields
+  end
+end
@@ -0,0 +1,13 @@
+class CreateCustomFormsPluginAnswers < ActiveRecord::Migration
+  def self.up
+    create_table :custom_forms_plugin_answers do |t|
+      t.text :value
+      t.references :field
+      t.references :submission
+    end
+  end
+
+  def self.down
+    drop_table :custom_forms_plugin_answers
+  end
+end
@@ -0,0 +1,15 @@
+class CreateCustomFormsPluginSubmissions < ActiveRecord::Migration
+  def self.up
+    create_table :custom_forms_plugin_submissions do |t|
+      t.string :author_name
+      t.string :author_email
+      t.references :profile
+      t.references :form
+      t.timestamps
+    end
+  end
+
+  def self.down
+    drop_table :custom_forms_plugin_submissions
+  end
+end
@@ -0,0 +1,21 @@
+require 'ext/role_assignment_trigger'
+
+class CustomFormsPlugin < Noosfero::Plugin
+
+  def self.plugin_name
+    "Custom Forms"
+  end
+
+  def self.plugin_description
+    _("Enables the creation of forms.")
+  end
+
+  def stylesheet?
+    true
+  end
+
+  def control_panel_buttons
+    {:title => _('Manage Forms'), :icon => 'custom-forms', :url => {:controller => 'custom_forms_plugin_myprofile'}}
+  end
+
+end
@@ -0,0 +1,14 @@
+class CustomFormsPlugin::Answer < Noosfero::Plugin::ActiveRecord
+  belongs_to :field, :class_name => 'CustomFormsPlugin::Field'
+  belongs_to :submission, :class_name => 'CustomFormsPlugin::Submission'
+
+  validates_presence_of :field
+  validate :value_mandatory, :if => 'field.present?'
+
+  def value_mandatory
+    if field.mandatory && value.blank?
+      errors.add(field.slug.to_sym, _("is mandatory.").fix_i18n)
+    end
+  end
+end
+
@@ -0,0 +1,16 @@
+class CustomFormsPlugin::Field < ActiveRecord::Base
+  set_table_name :custom_forms_plugin_fields
+
+  validates_presence_of :form, :name
+  validates_uniqueness_of :slug, :scope => :form_id
+
+  belongs_to :form, :class_name => 'CustomFormsPlugin::Form', :dependent => :destroy
+  has_many :answers, :class_name => 'CustomFormsPlugin::Answer'
+
+  serialize :choices, Hash
+
+  before_validation do |field|
+    field.slug = field.name.to_slug if field.name.present?
+  end
+end
+
@@ -0,0 +1,70 @@
+class CustomFormsPlugin::Form < Noosfero::Plugin::ActiveRecord
+  belongs_to :profile
+
+  has_many :fields, :class_name => 'CustomFormsPlugin::Field'
+  has_many :submissions, :class_name => 'CustomFormsPlugin::Submission'
+
+  serialize :access
+
+  validates_presence_of :profile, :name
+  validates_uniqueness_of :slug, :scope => :profile_id
+  validate :access_format
+
+  before_validation do |form|
+    form.slug = form.name.to_slug if form.name.present?
+    form.access = nil if form.access.blank?
+  end
+
+  named_scope :from, lambda {|profile| {:conditions => {:profile_id => profile.id}}}
+  named_scope :on_memberships, {:conditions => {:on_membership => true}}
+=begin
+  named_scope :accessible_to lambda do |profile| 
+    #TODO should verify is profile is associated with the form owner
+    profile_associated = ???
+    {:conditions => ["
+      access IS NULL OR
+      (access='logged' AND :profile_present) OR
+      (access='associated' AND :profile_associated) OR
+      :profile_id in access
+    ", {:profile_present => profile.present?, :profile_associated => ???, :profile_id => profile.id}]}
+  end
+=end
+
+  def expired?
+    (begining.present? && Time.now < begining) || (ending.present? && Time.now > ending)
+  end
+
+  def accessible_to(target)
+    return true if access.nil? || target == profile
+    return false if target.nil?
+    return true if access == 'logged'
+    return true if access == 'associated' && ((profile.organization? && profile.members.include?(target)) || (profile.person? && profile.friends.include?(target)))
+    return true if access.kind_of?(Integer) && target.id == access
+    return true if access.kind_of?(Array) && access.include?(target.id)
+  end
+
+  private
+
+  def access_format
+    if access.present?
+      if access.kind_of?(String)
+        if access != 'logged' && access != 'associated'
+          errors.add(:access, _('Invalid string format of access.'))
+        end
+      elsif access.kind_of?(Integer)
+        if !Profile.exists?(access)
+          errors.add(:access, _('There is no profile with the provided id.'))
+        end
+      elsif access.kind_of?(Array)
+        access.each do |value|
+          if !value.kind_of?(Integer) || !Profile.exists?(value)
+            errors.add(:access, _('There is no profile with the provided id.')) 
+            break
+          end
+        end
+      else
+        errors.add(:access, _('Invalid type format of access.'))
+      end
+    end
+  end
+end
@@ -0,0 +1,119 @@
+module CustomFormsPlugin::Helper
+  def access_text(form)
+    return _('Public') if form.access.nil?
+    return _('Logged users') if form.access == 'logged'
+    if form.access == 'associated'
+      return _('Members') if form.profile.organization? 
+      return _('Friends') if form.profile.person?
+    end
+    return _('Custom')
+  end
+
+  def period_range(form)
+    if form.begining.blank? && form.ending.blank?
+      _('Always')
+    elsif form.begining.present? && form.ending.blank?
+      ('From %s') % time_format(form.begining)
+    elsif form.begining.blank? && form.ending.present?
+      _('Until %s') % time_format(form.ending)
+    elsif form.begining.present? && form.ending.present?
+      _('From %s until %s') % [time_format(form.begining), time_format(form.ending)]
+    end
+  end
+
+  def time_format(time)
+    minutes = (time.min == 0) ? '' : ':%M'
+    hour = (time.hour == 0 && minutes.blank?) ? '' : ' %H'
+    h = hour.blank? ? '' : 'h'
+    time.strftime("%Y-%m-%d#{hour+minutes+h}")
+  end
+
+  # TODO add the custom option that should offer the user the hability to
+  # choose the profiles one by one, using something like tokeninput
+  def access_options(profile)
+    associated = profile.organization? ? _('Members') : _('Friends')
+    [
+      [_('Public'), nil         ],
+      [_('Logged users'), 'logged'    ],
+      [ associated, 'associated'],
+    ]
+  end
+
+  def type_options
+    [
+      [_('Text'),   'text_field'  ],
+      [_('Select'), 'select_field']
+    ]
+  end
+
+  def type_to_label(type)
+    map = {
+      'text_field' => _('Text'),
+      'select_field' => _('Select')
+    }
+    map[type_for_options(type)]
+  end
+
+  def type_for_options(type)
+    type.to_s.split(':').last.underscore
+  end
+
+  def display_custom_field(field, submission, form)
+    answer = submission.answers.select{|answer| answer.field == field}.first
+    field_tag = send("display_#{type_for_options(field.class)}",field, answer, form)
+    if field.mandatory? && !radio_button?(field) && !check_box?(field) && submission.id.nil?
+      required(labelled_form_field(field.name, field_tag))
+    else 
+      labelled_form_field(field.name, field_tag)
+    end
+  end
+
+  def display_text_field(field, answer, form)
+    value = answer.present? ? answer.value : field.default_value
+    text_field(form, field.name.to_slug, :value => value, :disabled => answer.present?)
+  end
+
+  def display_select_field(field, answer, form)
+    if field.list && field.multiple
+      selected = answer.present? ? answer.value.split(',') : []
+      select_tag "#{form}[#{field.name.to_slug}]", options_for_select(field.choices.to_a, selected), :multiple => true, :size => field.choices.size, :disabled => answer.present?
+    elsif !field.list && field.multiple
+      field.choices.map do |name, value|
+        default = answer.present? ? answer.value.split(',').include?(value) : false
+        labelled_check_box name, "#{form}[#{field.name.to_slug}][#{value}]", '1', default, :disabled => answer.present?
+      end.join("\n")
+    elsif field.list && !field.multiple
+      selected = answer.present? ? answer.value.split(',') : []
+      select_tag "#{form}[#{field.name.to_slug}]", options_for_select([['','']] + field.choices.to_a, selected), :disabled => answer.present?
+    elsif !field.list && !field.multiple
+      field.choices.map do |name, value|
+        default = answer.present? ? answer.value == value : true
+        labelled_radio_button name, "#{form}[#{field.name.to_slug}]", value, default, :disabled => answer.present?
+      end.join("\n")
+    end
+  end
+
+  def radio_button?(field)
+    type_for_options(field.class) == 'select_field' && !field.list && !field.multiple
+  end
+
+  def check_box?(field)
+    type_for_options(field.class) == 'select_field' && !field.list && field.multiple
+  end
+
+  def build_answers(submission, form)
+    answers = []
+    submission.each do |slug, value|
+      field = form.fields.select {|field| field.slug==slug}.first
+      if value.kind_of?(String)
+        final_value = value
+      elsif value.kind_of?(Array)
+        final_value = value.join(',')
+      elsif value.kind_of?(Hash)
+        final_value = value.map {|option, present| present == '1' ? option : nil}.compact.join(',')
+      end
+      answers << CustomFormsPlugin::Answer.new(:field => field, :value => final_value)
+    end
+    answers
+  end
+end
@@ -0,0 +1,47 @@
+class CustomFormsPlugin::MembershipSurvey < Task
+
+  settings_items :form_id, :submission
+  validates_presence_of :form_id
+
+  include CustomFormsPlugin::Helper
+
+  def perform
+    form = CustomFormsPlugin::Form.find(form_id)
+    answers = build_answers(submission, form)
+    s = CustomFormsPlugin::Submission.create!(:form => form, :profile => target)
+    answers.map {|answer| answer.submission = s; answer.save!}
+  end
+
+  def title
+    _("Membership survey")
+  end
+
+  def subject
+    nil
+  end
+
+  def linked_subject
+    nil
+  end
+
+  def information
+    {:message => _('%{requestor} wants you to fill in some information.')}
+  end
+
+  def accept_details
+    true
+  end
+
+  def icon
+    {:type => :profile_image, :profile => requestor, :url => requestor.url}
+  end
+
+  def target_notification_message
+    _('After joining %{requestor}, the administrators of this organization
+      wants you to fill in some further information.') % {:requestor => requestor.name}
+  end
+
+  def target_notification_description
+    _('%{requestor} wants to fill in some further information.') % {:requestor => requestor.name}
+  end
+end
@@ -0,0 +1,4 @@
+class CustomFormsPlugin::Numeric < CustomFormsPlugin::Field
+  set_table_name :custom_forms_plugin_fields
+end
+
@@ -0,0 +1,4 @@
+class CustomFormsPlugin::SelectField < CustomFormsPlugin::Field
+  set_table_name :custom_forms_plugin_fields
+  validates_presence_of :choices
+end
@@ -0,0 +1,12 @@
+class CustomFormsPlugin::Submission < Noosfero::Plugin::ActiveRecord
+  belongs_to :form, :class_name => 'CustomFormsPlugin::Form'
+  belongs_to :profile
+
+  has_many :answers, :class_name => 'CustomFormsPlugin::Answer'
+
+  validates_presence_of :form
+  validates_presence_of :author_name, :author_email, :if => lambda {|submission| submission.profile.nil?}
+  validates_uniqueness_of :author_email, :scope => :form_id, :allow_nil => true
+  validates_format_of :author_email, :with => Noosfero::Constants::EMAIL_FORMAT, :if => (lambda {|submission| !submission.author_email.blank?})
+end
+
@@ -0,0 +1,3 @@
+class CustomFormsPlugin::TextField < CustomFormsPlugin::Field
+  set_table_name :custom_forms_plugin_fields
+end
@@ -0,0 +1,30 @@
+module RoleAssignmentTrigger
+  def self.included(base)
+    base.class_eval do
+      before_create do |ra|
+        profile = ra.resource
+        person = ra.accessor
+        ok = !profile.nil? && !person.nil? && profile.environment.present?
+        if ok && profile.environment.plugin_enabled?(CustomFormsPlugin) && !person.is_member_of?(profile)
+          CustomFormsPlugin::Form.from(profile).on_memberships.each do |form|
+            CustomFormsPlugin::MembershipSurvey.create!(:requestor => profile, :target => person, :form_id => form.id)
+          end
+        end
+      end
+  
+      after_destroy do |ra|
+        profile = ra.resource
+        person = ra.accessor
+        ok = !profile.nil? && !person.nil? && profile.environment.present?
+        if ok && profile.environment.plugin_enabled?(CustomFormsPlugin) && !person.is_member_of?(profile)
+          CustomFormsPlugin::Form.from(profile).on_memberships.each do |form|
+            task = person.tasks.pending.select {|task| task.kind_of?(CustomFormsPlugin::MembershipSurvey) && task.form_id == form.id}.first
+            task.cancel if task
+          end
+        end
+      end
+    end
+  end
+end
+
+RoleAssignment.send :include, RoleAssignmentTrigger
@@ -0,0 +1,141 @@
+jQuery('.icon-edit').live('click', function() {
+  elem = this;
+  jQuery.fn.colorbox({
+    overlayClose: false,
+    escKey: false,
+    inline: true,
+    href: function(){
+      id = jQuery(elem).attr('field_id');
+      type = jQuery('#fields_'+id+'_type').val().split('_')[0];
+      selector = '#edit-'+type+'-'+id
+      jQuery(selector).show();
+      return selector
+    }
+  });
+  return false;
+});
+
+jQuery('.remove-field').live('click', function(){
+  id = jQuery(this).attr('field_id');
+  jQuery('#field-'+id).slideDown(function(){
+    jQuery('#field-'+id).remove();
+  });
+  return false
+});
+
+jQuery('.remove-option').live('click', function(){
+  field_id = jQuery(this).attr('field_id');
+  option_id = jQuery(this).attr('option_id');
+  selector = '#field-'+field_id+'-option-'+option_id
+  jQuery(selector).slideDown(function(){
+    jQuery(selector).remove();
+    jQuery.colorbox.resize();
+  });
+  return false
+});
+
+function updateEditText(id){
+  new_id = id+1
+  jQuery('#edit-text-'+id).attr('id', 'edit-text-'+new_id);
+  input = jQuery('#edit-text-'+new_id+' input');
+  jQuery('#edit-text-'+new_id+' .colorbox-ok-button').attr('div_id', 'edit-text-'+new_id);
+  input.attr('id', input.attr('id').replace(id,new_id));
+  input.attr('name', input.attr('name').replace(id,new_id));
+  label = jQuery('#edit-text-'+new_id+' label');
+  label.attr('for', label.attr('for').replace(id,new_id));
+}
+
+function updateEditSelect(id){
+  new_id = id+1
+  jQuery('#edit-select-'+id).attr('id', 'edit-select-'+new_id);
+  jQuery('#edit-select-'+new_id+' .colorbox-ok-button').attr('div_id', 'edit-select-'+new_id);
+  jQuery('tr[id^=field-'+id+'-option').each(function(id, element){
+    jQuery(element).attr('id', jQuery(element).attr('id').replace('field-'+id,'field-'+new_id));
+  });
+  jQuery('#edit-select-'+new_id+' label').each(function(index, element){
+    label = jQuery(element);
+    label.attr('for', label.attr('for').replace(id,new_id));
+  });
+  jQuery('#edit-select-'+new_id+' input').each(function(index, element){
+    input = jQuery(element);
+    input.attr('id', input.attr('id').replace(id,new_id));
+    input.attr('name', input.attr('name').replace(id,new_id));
+  });
+  jQuery('#edit-select-'+new_id+' .remove-option').each(function(index, element){
+    jQuery(element).attr('field_id',new_id);
+  });
+  jQuery('#edit-select-'+new_id+' .new-option').attr('field_id',new_id);
+  jQuery('#edit-select-'+new_id+' #empty-option-'+id).attr('id','empty-option-'+new_id);
+}
+
+function updateEmptyField(id){
+  id = parseInt(id);
+  empty_field = jQuery('#empty-field');
+  empty_field.attr('last_id', (id + 1).toString());
+  jQuery('#empty-field input').each(function(index, element){
+    new_id = jQuery(element).attr('id').replace(id,id+1);
+    jQuery(element).attr('id', new_id);
+    new_name = jQuery(element).attr('name').replace(id,id+1);
+    jQuery(element).attr('name', new_name);
+  });
+  jQuery('#empty-field select').each(function(index, element){
+    new_id = jQuery(element).attr('id').replace(id,id+1);
+    jQuery(element).attr('id', new_id);
+    new_name = jQuery(element).attr('name').replace(id,id+1);
+    jQuery(element).attr('name', new_name);
+  });
+  jQuery('#empty-field a').each(function(index, element){
+    jQuery(element).attr('field_id', id+1);
+  });
+  updateEditText(id);
+  updateEditSelect(id);
+}
+
+function updateEmptyOption(field_id, option_id){
+  field_id = parseInt(field_id);
+  option_id = parseInt(option_id);
+  new_option_id = option_id+1;
+  empty_option = jQuery('#empty-option-'+field_id);
+  empty_option.attr('option_id',new_option_id);
+  jQuery('#empty-option-'+field_id+' .remove-option').attr('option_id', new_option_id);
+
+  name_id = ' #fields_'+field_id+'_choices_'+option_id+'_name';
+  jQuery('#empty-option-'+field_id+name_id).attr('name', 'fields['+field_id+'][choices]['+new_option_id+'][name]');
+  jQuery('#empty-option-'+field_id+name_id).attr('id', 'fields_'+field_id+'_choices_'+new_option_id+'_name');
+
+  value_id = ' #fields_'+field_id+'_choices_'+option_id+'_value';
+  jQuery('#empty-option-'+field_id+value_id).attr('name', 'fields['+field_id+'][choices]['+new_option_id+'][value]');
+  jQuery('#empty-option-'+field_id+value_id).attr('id', 'fields_'+field_id+'_choices_'+new_option_id+'_value');
+}
+
+jQuery('#new-field').live('click', function(){
+  empty_field = jQuery('#empty-field');
+  id = empty_field.attr('last_id');
+  edit_text = jQuery('#edit-text-'+id);
+  edit_select = jQuery('#edit-select-'+id);
+  new_field = empty_field.clone();
+  new_field.attr('id','field-'+id);
+  new_field.insertBefore(empty_field).slideDown();
+  edit_text.clone().insertAfter(edit_text);
+  edit_select.clone().insertAfter(edit_select);
+  updateEmptyField(id);
+  return false
+});
+
+jQuery('.new-option').live('click', function(){
+  field_id = jQuery(this).attr('field_id');
+  empty_option = jQuery('#empty-option-'+field_id);
+  option_id = empty_option.attr('option_id');
+  new_option = empty_option.clone();
+  new_option.attr('id','field-'+field_id+'-option-'+option_id);
+  new_option.insertBefore(empty_option).slideDown();
+  jQuery.colorbox.resize();
+  updateEmptyOption(field_id, option_id);
+  return false
+});
+
+jQuery('.colorbox-ok-button').live('click', function(){
+  jQuery('#'+jQuery(this).attr('div_id')).hide();
+  jQuery.colorbox.close();
+  return false
+});
@@ -0,0 +1,30 @@
+.controller-profile_editor a.control-panel-custom-forms,
+.controller-profile_editor .msie6 a.control-panel-custom-forms {
+  background-image: url(/plugins/custom_forms/icons/custom-forms.png)
+}
+
+.action-table {
+  width: 100%; 
+  overflow: hidden;
+}
+
+.action-table th,
+.action-table td{
+  text-align: center;
+}
+
+.action-table td{
+  cursor: move;
+}
+
+.action-table .actions{
+  white-space: nowrap;
+}
+
+.action-table .new-item{
+  background-color: #EEE;
+}
+
+.edit-information {
+  display: none;
+}
@@ -0,0 +1,114 @@
+require File.dirname(__FILE__) + '/../../../../test/test_helper'
+require File.dirname(__FILE__) + '/../../controllers/custom_forms_plugin_myprofile_controller'
+
+# Re-raise errors caught by the controller.
+class CustomFormsPluginMyprofileController; def rescue_action(e) raise e end; end
+
+class CustomFormsPluginMyprofileControllerTest < ActionController::TestCase
+  def setup
+    @controller = CustomFormsPluginMyprofileController.new
+    @request    = ActionController::TestRequest.new
+    @response   = ActionController::TestResponse.new
+    @profile = create_user('profile').person
+    login_as(@profile.identifier)
+    environment = Environment.default
+    environment.enable_plugin(CustomFormsPlugin)
+  end
+
+  attr_reader :profile
+
+  should 'list forms associated with profile' do
+    another_profile = fast_create(Profile)
+    f1 = CustomFormsPlugin::Form.create!(:profile => profile, :name => 'Free Software')
+    f2 = CustomFormsPlugin::Form.create!(:profile => profile, :name => 'Open Source')
+    f3 = CustomFormsPlugin::Form.create!(:profile => another_profile, :name => 'Open Source')
+
+    get :index, :profile => profile.identifier
+
+    assert_includes assigns(:forms), f1
+    assert_includes assigns(:forms), f2
+    assert_not_includes assigns(:forms), f3
+  end
+
+  should 'destroy form' do
+    form = CustomFormsPlugin::Form.create!(:profile => profile, :name => 'Free Software')
+    assert CustomFormsPlugin::Form.exists?(form.id)
+    post :remove, :profile => profile.identifier, :id => form.id
+    assert !CustomFormsPlugin::Form.exists?(form.id)
+  end
+
+  should 'create a form' do
+    format = '%Y-%m-%d %H:%M'
+    begining = Time.now.strftime(format)
+    ending = (Time.now + 1.day).strftime('%Y-%m-%d %H:%M')
+    assert_difference CustomFormsPlugin::Form, :count, 1 do
+      post :create, :profile => profile.identifier,
+        :form => {
+          :name => 'My Form',
+          :access => 'logged',
+          :begining => begining,
+          :ending => ending,
+          :description => 'Cool form'},
+        :fields => {
+          1 => {
+            :name => 'Name',
+            :default_value => 'Jack',
+            :type => 'text_field'
+          },
+          2 => {
+            :name => 'Color',
+            :list => '1',
+            :type => 'select_field',
+            :choices => {
+              1 => {:name => 'Red', :value => 'red'},
+              2 => {:name => 'Blue', :value => 'blue'},
+              3 => {:name => 'Black', :value => 'black'}
+            }
+          }
+        }
+    end
+
+    form = CustomFormsPlugin::Form.find_by_name('My Form')
+    assert_equal 'logged', form.access
+    assert_equal begining, form.begining.strftime(format)
+    assert_equal ending, form.ending.strftime(format)
+    assert_equal 'Cool form', form.description
+    assert_equal 2, form.fields.count
+
+    f1 = form.fields.first
+    f2 = form.fields.last
+
+    assert_equal 'Name', f1.name
+    assert_equal 'Jack', f1.default_value
+    assert f1.kind_of?(CustomFormsPlugin::TextField)
+
+    assert_equal 'Color', f2.name
+    assert_equal 'red', f2.choices['Red']
+    assert_equal 'blue', f2.choices['Blue']
+    assert_equal 'black', f2.choices['Black']
+    assert f2.list
+    assert f2.kind_of?(CustomFormsPlugin::SelectField)
+  end
+
+  should 'edit a form' do
+    form = CustomFormsPlugin::Form.create!(:profile => profile, :name => 'Free Software')
+    field = CustomFormsPlugin::TextField.create!(:form => form, :name => 'License')
+    format = '%Y-%m-%d %H:%M'
+    begining = Time.now.strftime(format)
+    ending = (Time.now + 1.day).strftime('%Y-%m-%d %H:%M')
+
+    post :edit, :profile => profile.identifier, :id => form.id,
+      :form => {:name => 'My Form', :access => 'logged', :begining => begining, :ending => ending, :description => 'Cool form'},
+      :fields => {1 => {:real_id => field.id.to_s, :name => 'Source'}}
+
+    form.reload
+    field.reload
+
+    assert_equal 'logged', form.access
+    assert_equal begining, form.begining.strftime(format)
+    assert_equal ending, form.ending.strftime(format)
+    assert_equal 'Cool form', form.description
+    assert_equal 'Source', field.name
+  end
+end
+
@@ -0,0 +1,38 @@
+require File.dirname(__FILE__) + '/../../../../../test/test_helper'
+
+class CustomFormsPlugin::AnswerTest < ActiveSupport::TestCase
+  should 'validates presence of field' do
+    answer = CustomFormsPlugin::Answer.new
+    answer.valid?
+    assert answer.errors.invalid?(:field)
+
+    form = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => fast_create(Profile))
+    field = CustomFormsPlugin::Field.create!(:name => 'License', :form => form)
+    answer.field = field
+    answer.valid?
+    assert !answer.errors.invalid?(:field)
+  end
+
+  should 'belong to a submission' do
+    form = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => fast_create(Profile))
+    submission = CustomFormsPlugin::Submission.create!(:form => form, :profile => fast_create(Profile))
+    answer = CustomFormsPlugin::Answer.new
+    answer.submission = submission
+
+    assert_equal submission, answer.submission
+  end
+
+  should 'require presence of value if field is mandatory' do
+    form = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => fast_create(Profile))
+    field = CustomFormsPlugin::Field.create!(:name => 'License', :form => form, :mandatory => true)
+    answer = CustomFormsPlugin::Answer.new(:field => field)
+    answer.valid?
+    assert answer.errors.invalid?(field.slug.to_sym)
+
+    answer.value = "GPL"
+    answer.valid?
+    assert !answer.errors.invalid?(field.slug.to_sym)
+  end
+
+end
+
@@ -0,0 +1,64 @@
+require File.dirname(__FILE__) + '/../../../../../test/test_helper'
+
+class CustomFormsPlugin::FieldTest < ActiveSupport::TestCase
+  should 'validate presence of form' do
+    field = CustomFormsPlugin::Field.new
+    field.valid?
+    assert field.errors.invalid?(:form)
+    assert field.errors.invalid?(:name)
+
+    form = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => fast_create(Profile))
+    field.form = form
+    field.name = 'License'
+    field.valid?
+    assert !field.errors.invalid?(:form)
+    assert !field.errors.invalid?(:name)
+  end
+
+  should 'set slug before validation based on name' do
+    field = CustomFormsPlugin::Field.new(:name => 'Name')
+    field.valid?
+    assert_equal field.name.to_slug, field.slug
+  end
+
+  should 'validate uniqueness of slug scoped on the form' do
+    form1 = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => fast_create(Profile))
+    form2 = CustomFormsPlugin::Form.create!(:name => 'Open Source', :profile => fast_create(Profile))
+    f1 = CustomFormsPlugin::Field.create!(:name => 'License', :form => form1)
+    f2 = CustomFormsPlugin::Field.new(:name => 'License', :form => form1)
+    f3 = CustomFormsPlugin::Field.new(:name => 'License', :form => form2)
+
+    f2.valid?
+    f3.valid?
+
+    assert f2.errors.invalid?(:slug)
+    assert !f3.errors.invalid?(:slug)
+  end
+
+  should 'set mandatory field as false by default' do
+    field = CustomFormsPlugin::Field.new
+    assert !field.mandatory
+  end
+
+  should 'have answers' do
+    form = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => fast_create(Profile))
+    field = CustomFormsPlugin::Field.create!(:name => 'License', :form => form)
+    a1 = CustomFormsPlugin::Answer.create!(:field => field)
+    a2 = CustomFormsPlugin::Answer.create!(:field => field)
+
+    assert_includes field.answers, a1
+    assert_includes field.answers, a2
+  end
+
+  should 'serialize choices into a hash' do
+    form = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => fast_create(Profile))
+    field = CustomFormsPlugin::Field.create!(:name => 'License', :form => form)
+    field.choices = {'First' => 1, 'Second' => 2, 'Third' => 3}
+    field.save!
+
+    assert_equal 1, field.choices['First']
+    assert_equal 2, field.choices['Second']
+    assert_equal 3, field.choices['Third']
+  end
+end
+
@@ -0,0 +1,172 @@
+require File.dirname(__FILE__) + '/../../../../../test/test_helper'
+
+class CustomFormsPlugin::FormTest < ActiveSupport::TestCase
+  should 'validates presence of a profile and a name' do
+    form = CustomFormsPlugin::Form.new
+    form.valid?
+    assert form.errors.invalid?(:profile)
+    assert form.errors.invalid?(:name)
+
+    form.profile = fast_create(Profile)
+    form.name = 'Free Software'
+    form.valid?
+    assert !form.errors.invalid?(:profile)
+    assert !form.errors.invalid?(:name)
+  end
+
+  should 'have many fields including fields subclasses' do
+    form = CustomFormsPlugin::Form.create!(:profile => fast_create(Profile), :name => 'Free Software')
+    f1 = CustomFormsPlugin::Field.create!(:form => form, :name => 'License')
+    f2 = CustomFormsPlugin::Field.create!(:form => form, :name => 'Code')
+    f3 = CustomFormsPlugin::TextField.create!(:form => form, :name => 'Developer')
+
+    assert_includes form.fields, f1
+    assert_includes form.fields, f2
+    assert_includes form.fields, f3
+  end
+
+  should 'have many submissions' do
+    form = CustomFormsPlugin::Form.create!(:profile => fast_create(Profile), :name => 'Free Software')
+    s1 = CustomFormsPlugin::Submission.create!(:form => form, :profile => fast_create(Profile))
+    s2 = CustomFormsPlugin::Submission.create!(:form => form, :profile => fast_create(Profile))
+
+    assert_includes form.submissions, s1
+    assert_includes form.submissions, s2
+  end
+
+  should 'set slug before validation based on name' do
+    form = CustomFormsPlugin::Form.new(:name => 'Name')
+    form.valid?
+    assert_equal form.name.to_slug, form.slug
+  end
+
+  should 'validates uniqueness of slug scoped on profile' do
+    profile = fast_create(Profile)
+    another_profile = fast_create(Profile)
+    CustomFormsPlugin::Form.create!(:profile => profile, :name => 'Free Software')
+    form = CustomFormsPlugin::Form.new(:profile => profile, :name => 'Free Software')
+    form.valid?
+    assert form.errors.invalid?(:slug)
+
+    form.profile = another_profile
+    form.valid?
+    assert !form.errors.invalid?(:slug)
+  end
+
+  should 'define form expiration' do
+    form = CustomFormsPlugin::Form.new
+    assert !form.expired?
+
+    form.begining = Time.now + 1.day
+    assert form.expired?
+
+    form.begining = Time.now - 1.day
+    assert !form.expired?
+
+    form.begining = nil
+    form.ending = Time.now + 1.day
+    assert !form.expired?
+
+    form.ending = Time.now - 1.day
+    assert form.expired?
+
+    form.begining = Time.now - 1.day
+    form.ending = Time.now + 1.day
+    assert !form.expired?
+  end
+
+  should 'validates format of access' do
+    form = CustomFormsPlugin::Form.new
+    form.valid?
+    assert !form.errors.invalid?(:access)
+
+    form.access = 'bli'
+    form.valid?
+    assert form.errors.invalid?(:access)
+
+    form.access = 'logged'
+    form.valid?
+    assert !form.errors.invalid?(:access)
+
+    form.access = 'associated'
+    form.valid?
+    assert !form.errors.invalid?(:access)
+
+    form.access = {:bli => 1}
+    form.valid?
+    assert form.errors.invalid?(:access)
+
+    form.access = 999
+    form.valid?
+    assert form.errors.invalid?(:access)
+
+    p1 = fast_create(Profile)
+    form.access = p1.id
+    form.valid?
+    assert !form.errors.invalid?(:access)
+
+    p2 = fast_create(Profile)
+    p3 = fast_create(Profile)
+    form.access = [p1,p2,p3].map(&:id)
+    form.valid?
+    assert !form.errors.invalid?(:access)
+  end
+
+  should 'defines who is able to access the form' do
+    owner = fast_create(Community)
+    form = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => owner)
+    assert form.accessible_to(nil)
+
+    form.access = 'logged'
+    assert !form.accessible_to(nil)
+    person = fast_create(Person)
+    assert form.accessible_to(person)
+
+    form.access = 'associated'
+    assert !form.accessible_to(person)
+    owner.add_member(person)
+    assert form.accessible_to(person)
+
+    p1 = fast_create(Profile)
+    form.access = p1.id
+    assert !form.accessible_to(person)
+    assert form.accessible_to(p1)
+
+    p2 = fast_create(Profile)
+    form.access = [person.id, p1.id]
+    assert form.accessible_to(person)
+    assert form.accessible_to(p1)
+    assert !form.accessible_to(p2)
+    form.access << p2.id
+    assert form.accessible_to(p2)
+
+    assert form.accessible_to(owner)
+  end
+
+  should 'have a named_scope that retrieve forms from a profile' do
+    profile = fast_create(Profile)
+    another_profile = fast_create(Profile)
+    f1 = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => profile)
+    f2 = CustomFormsPlugin::Form.create!(:name => 'Open Source', :profile => profile)
+    f3 = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => another_profile)
+    scope = CustomFormsPlugin::Form.from(profile)
+
+    assert_equal ActiveRecord::NamedScope::Scope, scope.class
+    assert_includes scope, f1
+    assert_includes scope, f2
+    assert_not_includes scope, f3
+  end
+
+  should 'have a named_scope that retrieves all forms that are triggered on membership' do
+    profile = fast_create(Profile)
+    f1 = CustomFormsPlugin::Form.create!(:name => 'On membership 1', :profile => profile, :on_membership => true)
+    f2 = CustomFormsPlugin::Form.create!(:name => 'On membership 2', :profile => profile, :on_membership => true)
+    f3 = CustomFormsPlugin::Form.create!(:name => 'Not on memberhsip', :profile => profile, :on_membership => false)
+    scope = CustomFormsPlugin::Form.from(profile).on_memberships
+
+    assert_equal ActiveRecord::NamedScope::Scope, scope.class
+    assert_includes scope, f1
+    assert_includes scope, f2
+    assert_not_includes scope, f3
+  end
+end
@@ -0,0 +1,31 @@
+require File.dirname(__FILE__) + '/../../../../../test/test_helper'
+
+class CustomFormsPlugin::MembershipSurveyTest < ActiveSupport::TestCase
+  should 'validates presence of form_id' do
+    task = CustomFormsPlugin::MembershipSurvey.new
+    task.valid?
+    assert task.errors.invalid?(:form_id)
+
+    task.form_id = 1
+    task.valid?
+    assert !task.errors.invalid?(:form_id)
+  end
+
+  should 'create submission with answers on perform' do
+    profile = fast_create(Profile)
+    person = fast_create(Person)
+    form = CustomFormsPlugin::Form.create!(:name => 'Simple Form', :profile => profile)
+    field = CustomFormsPlugin::Field.create!(:name => 'Name', :form => form)
+    task = CustomFormsPlugin::MembershipSurvey.create!(:form_id => form.id, :submission => {'name' => 'Jack'}, :target => person, :requestor => profile)
+
+    assert_difference CustomFormsPlugin::Submission, :count, 1 do
+      task.finish
+    end
+
+    submission = CustomFormsPlugin::Submission.last
+    assert_equal submission.answers.count, 1 
+
+    answer = submission.answers.first
+    assert_equal answer.value, 'Jack'
+  end
+end
@@ -0,0 +1,13 @@
+require File.dirname(__FILE__) + '/../../../../../test/test_helper'
+
+class CustomFormsPlugin::SelectFieldTest < ActiveSupport::TestCase
+  should 'validate presence of choices, multiple and list' do
+    select = CustomFormsPlugin::SelectField.new
+    select.valid?
+    assert select.errors.invalid?(:choices)
+
+    select.choices = {'label' => 'value'}
+    select.valid?
+    assert !select.errors.invalid?(:choices)
+  end
+end
@@ -0,0 +1,46 @@
+require File.dirname(__FILE__) + '/../../../../../test/test_helper'
+
+class CustomFormsPlugin::SubmissionTest < ActiveSupport::TestCase
+  should 'validates presence of form' do
+    submission = CustomFormsPlugin::Submission.new
+    submission.valid?
+    assert submission.errors.invalid?(:form)
+
+    form = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => fast_create(Profile))
+    submission.form = form
+    submission.valid?
+    assert !submission.errors.invalid?(:form)
+  end
+
+  should 'belong to a profile' do
+    profile = fast_create(Profile)
+    submission = CustomFormsPlugin::Submission.new
+    submission.profile = profile
+    assert_equal profile, submission.profile
+  end
+
+  should 'require presence of author name and email if profile is nil' do
+    submission = CustomFormsPlugin::Submission.new
+    submission.valid?
+    assert submission.errors.invalid?(:author_name)
+    assert submission.errors.invalid?(:author_email)
+
+    submission.author_name = 'Jack Sparrow'
+    submission.author_email = 'jack@black-pearl.com'
+    submission.valid?
+    assert !submission.errors.invalid?(:author_name)
+    assert !submission.errors.invalid?(:author_email)
+  end
+
+  should 'have answers' do
+    form = CustomFormsPlugin::Form.create!(:name => 'Free Software', :profile => fast_create(Profile))
+    field = CustomFormsPlugin::Field.create!(:name => 'License', :form => form)
+    submission = CustomFormsPlugin::Submission.create!(:form => form, :profile => fast_create(Profile))
+    a1 = CustomFormsPlugin::Answer.create!(:field => field, :submission => submission)
+    a2 = CustomFormsPlugin::Answer.create!(:field => field, :submission => submission)
+
+    assert_includes submission.answers, a1
+    assert_includes submission.answers, a2
+  end
+end
+
@@ -0,0 +1,51 @@
+require File.dirname(__FILE__) + '/../../../../../test/test_helper'
+
+class RoleAssignmentsTest < ActiveSupport::TestCase
+  should 'create membership_surveys on membership creation' do
+    environment = Environment.default
+    environment.enable_plugin(CustomFormsPlugin)
+    organization = fast_create(Organization)
+    person = fast_create(Person)
+    f1 = CustomFormsPlugin::Form.create!(:profile => organization, :name => 'Form 1', :on_membership => true)
+    f2 = CustomFormsPlugin::Form.create!(:profile => organization, :name => 'Form 2', :on_membership => true)
+    f3 = CustomFormsPlugin::Form.create!(:profile => organization, :name => 'Form 3', :on_membership => false)
+
+    assert_difference CustomFormsPlugin::MembershipSurvey, :count, 2 do
+      organization.add_member(person)
+    end
+  end
+
+  should 'create membership_survey on membership creation with form accessible to members only' do
+    environment = Environment.default
+    environment.enable_plugin(CustomFormsPlugin)
+    organization = fast_create(Organization)
+    person = fast_create(Person)
+    form = CustomFormsPlugin::Form.create!(:profile => organization, :name => 'Form', :on_membership => true, :access => 'associated')
+
+    assert_difference CustomFormsPlugin::MembershipSurvey, :count, 1 do
+      organization.add_member(person)
+    end
+  end
+
+  should 'cancel membership_surveys if membership is undone and task is active' do
+    environment = Environment.default
+    environment.enable_plugin(CustomFormsPlugin)
+    organization = fast_create(Organization)
+    person = fast_create(Person)
+    form = CustomFormsPlugin::Form.create!(:profile => organization, :name => 'Form', :on_membership => true)
+    organization.add_member(person)
+
+    assert_difference CustomFormsPlugin::MembershipSurvey.pending, :count, -1 do
+      organization.remove_member(person)
+    end
+
+    organization.add_member(person)
+    task = CustomFormsPlugin::MembershipSurvey.last
+    task.status = Task::Status::FINISHED
+    task.save!
+    assert_no_difference CustomFormsPlugin::MembershipSurvey.finished, :count do
+      organization.remove_member(person)
+    end
+  end
+end
+
@@ -0,0 +1,33 @@
+<% elem_id = "edit-select-#{counter}" %>
+<div id=<%= elem_id %> class='edit-information'>
+  <h2><%= _('Options') %></h2>
+  <table class='action-table' style='width: 420px'>
+    <tr>
+      <th style='width: 40%'><%= _('Name') %></th>
+      <th style='width: 40%'><%= _('Value') %></th>
+      <th style='width: 20%'><%= _('Actions') %></th>
+    </tr>
+    <% option_counter = 1 %>
+    <% (field.choices || {}).each do |name, value| %>
+      <%= render :partial => 'option', :locals => {:name => name, :value => value, :counter => counter, :option_counter => option_counter} %>
+      <% option_counter += 1 %>
+    <% end %>
+    <%= render :partial => 'empty_option', :locals => {:counter => counter, :option_counter => option_counter} %>
+    <tr class='new-item'>
+      <td colspan='3'>
+        <%= button(:add, _('Add a new option'), '#', :class => 'new-option', :field_id => counter)%>
+      </td>
+    </tr>
+  </table>
+
+  <h3><%= _('Type') %></h3>
+  <%= labelled_radio_button 'Radio', "fields[#{counter}][kind]", 'radio', !field.multiple && !field.list %><br />
+  <%= labelled_radio_button 'Checkbox', "fields[#{counter}][kind]", 'check_box', field.multiple && !field.list %><br />
+  <%= labelled_radio_button 'Select', "fields[#{counter}][kind]", 'select', !field.multiple && field.list %><br />
+  <%= labelled_radio_button 'Multiple Select', "fields[#{counter}][kind]", 'multiple_select', field.multiple && field.list %><br />
+
+  <% button_bar do %>
+    <%= button :ok, _('Ok'), '#', :class => 'colorbox-ok-button', :div_id => elem_id %>
+  <% end %>
+</div>
+
@@ -0,0 +1,10 @@
+<% elem_id = "edit-text-#{counter}" %>
+<div id=<%= elem_id %> class='edit-information'>
+  <h2><%= _('Default value') %></h2>
+  <%= labelled_form_field _('Default value'), text_field("fields[#{counter}]", :default_value, :value => field.default_value) %>
+
+  <% button_bar do %>
+    <%= button :ok, _('Ok'), '#', :class => 'colorbox-ok-button', :div_id => elem_id %>
+  <% end %>
+</div>
+
@@ -0,0 +1,10 @@
+<tr id="empty-field" style='display: none' last_id=<%= counter %>>
+  <td style="text-align: left"><%= text_field "fields[#{counter}]", :name, :size => 25 %></td>
+  <td><%= select "fields[#{counter}]", :type, type_options, :selected => type_for_options(field.class) %></td>
+  <td><%= check_box "fields[#{counter}]", :mandatory %></td>
+  <%= hidden_field "fields[#{counter}]", :form_id, :value => @form.id %>
+  <td class='actions'>
+    <%= button_without_text :edit, _('Edit'), '', :field_id => counter %>
+    <%= button_without_text :remove, _('Remove'), '#', :class => 'remove-field', :field_id => counter,  :confirm => _('Are you sure you want to remove this field?') %>
+  </td>
+</tr>
@@ -0,0 +1,8 @@
+<tr id=<%= "empty-option-#{counter}" %> option_id=<%= option_counter %> style="display: none;">
+  <td><%= text_field_tag("fields[#{counter}][choices][#{option_counter}][name]") %></td>
+  <td><%= text_field_tag("fields[#{counter}][choices][#{option_counter}][value]", nil, :size => 15) %></td>
+  <td class='actions'>
+    <%= button_without_text :remove, _('Remove'), '#', :class => 'remove-option', :field_id => counter, :option_id => option_counter,  :confirm => _('Are you sure you want to remove this option?') %>
+  </td>
+</tr>
+
@@ -0,0 +1,12 @@
+<tr id=<%= "field-#{counter}" %>>
+  <td style="text-align: left"><%= text_field "fields[#{counter}]", :name, :value => field.name, :size => 25 %></td>
+  <td><%= type_to_label(field.type) %></td>
+  <%= hidden_field "fields[#{counter}]", :type, :value => type_for_options(field.class) %>
+  <td><%= check_box "fields[#{counter}]", :mandatory, :checked => field.mandatory %></td>
+  <%= hidden_field "fields[#{counter}]", :real_id, :value => field.id %>
+  <%= hidden_field "fields[#{counter}]", :form_id, :value => @form.id %>
+  <td class='actions'>
+    <%= button_without_text :edit, _('Edit'), '#', :field_id => counter %>
+    <%= button_without_text :remove, _('Remove'), '#', :class => 'remove-field', :field_id => counter,  :confirm => _('Are you sure you want to remove this field?') %>
+  </td>
+</tr>
@@ -0,0 +1,56 @@
+<% self.extend(CustomFormsPlugin::Helper) %>
+
+<%= error_messages_for :form %>
+
+<% form_for :form, @form do |f| %>
+  <%= required labelled_form_field _('Name'), f.text_field(:name) %>
+  <%= labelled_form_field(_('Período'), (
+    date_range_field('form[begining]', 'form[ending]', @form.begining, @form.ending, 
+                     '%Y-%m-%d %H:%M', 
+                     { :time => true, :change_month => true, :change_year => true,
+                       :date_format => 'yy-mm-dd', :time_format => 'hh:mm' },
+                     { :size => 14 })
+  )) %>
+  <%= labelled_form_field _('Access'), f.select(:access, access_options(profile))%>
+  <% if profile.organization? %>
+    <%= labelled_form_field _('Triggered on membership'), f.check_box(:on_membership) %>
+  <% end %>
+  <%= labelled_form_field _('Description'), f.text_area(:description, :style => 'width: 100%') %>
+
+  <h2><%= _('Fields') %></h2>
+  <table class="action-table" id='fields-table'>
+    <tr>
+      <th style='width: 40%'><%= _('Name') %></th>
+      <th style='width: 30%'><%= _('Type') %></th>
+      <th style='width: 10%'><%= _('Mandatory') %></th>
+      <th style='width: 20%'><%= _('Actions') %></th>
+    </tr>
+    <% counter = 1 %>
+    <% @fields.each do |field| %>
+      <%= render :partial => 'field', :locals => {:field => field, :counter => counter} %>
+      <% counter += 1 %>
+    <% end %>
+    <%= render :partial => 'empty_field', :locals => {:field => @empty_field, :counter => counter} %>
+    <tr class='new-item'>
+      <td colspan='5'>
+        <%= button(:add, _('Add a new field'), '#', :id => 'new-field')%>
+      </td>
+    </tr>
+  </table>
+
+  <% counter = 1 %>
+  <% @fields.each do |field| %>
+    <%= render :partial => 'edit_text', :locals => {:field => field, :counter => counter} %>
+    <%= render :partial => 'edit_select', :locals => {:field => field, :counter => counter} %>
+    <% counter += 1 %>
+  <% end %>
+
+  <%= render :partial => 'edit_text', :locals => {:field => @empty_field, :counter => counter} %>
+  <%= render :partial => 'edit_select', :locals => {:field => @empty_field, :counter => counter} %>
+
+  <% button_bar do %>
+    <%= submit_button :save, _('Save'), :cancel => {:action => 'index'}%>
+  <% end %>
+<% end %>
+
+<%= javascript_include_tag '../plugins/custom_forms/field' %>
@@ -0,0 +1,7 @@
+<tr id=<%= "field-#{counter}-option-#{option_counter}" %> style="display: auto;">
+  <td><%= text_field_tag("fields[#{counter}][choices][#{option_counter}][name]", name) %></td>
+  <td><%= text_field_tag("fields[#{counter}][choices][#{option_counter}][value]", value) %></td>
+  <td class='actions'>
+    <%= button_without_text :remove, _('Remove'), '#', :class => 'remove-option', :field_id => counter, :option_id => option_counter,  :confirm => _('Are you sure you want to remove this option?') %>
+  </td>
+</tr>
@@ -0,0 +1,2 @@
+<h1><%= _('New form') %></h1>
+<%= render :partial => 'form' %>
@@ -0,0 +1,2 @@
+<h1><%= _('Edit form') %></h1>
+<%= render :partial => 'form' %>
@@ -0,0 +1,28 @@
+<% self.extend CustomFormsPlugin::Helper %>
+
+<h1><%= _('Manage forms') %></h1>
+<table class="action-table">
+  <tr>
+    <th style='width: 40%'><%= _('Name') %></th>
+    <th style='width: 30%'><%= _('Period') %></th>
+    <th style='width: 10%'><%= _('Submissions') %></th>
+    <th style='width: 10%'><%= _('Access') %></th>
+    <th style='width: 10%'><%= _('Actions') %></th>
+  </tr>
+  <% @forms.each do |form| %>
+    <tr>
+      <td><%= link_to form.name, {:controller => 'custom_forms_plugin_profile', :action => 'show', :id => form.id} %></td>
+      <td><%= period_range(form) %></td>
+      <td><%= form.submissions.count > 0 ? link_to(form.submissions.count, {:action => 'submissions', :id => form.id}) : 0 %></td>
+      <td><%= access_text(form) %></td>
+      <td class="actions">
+        <%= button_without_text :edit, _('Edit'), :action => 'edit', :id => form.id %>
+        <%= button_without_text :remove, _('Remove'), {:action => 'remove', :id => form.id}, :confirm => _('Are you sure you want to remove this form?') %></td>
+    </tr>
+  <% end %>
+  <tr id="new-item">
+    <td colspan='5'>
+      <%= button(:add, _('Add a new form'), :action => 'create')%>
+   </td>
+  </tr>
+</table>
@@ -0,0 +1,10 @@
+<h1><%= @form.name %></h1>
+<p><%= @form.description %></p>
+
+<% fields_for :submission, @submission do |f| %>
+  <%= render :partial => 'shared/form_submission', :locals => {:f => f} %>
+<% end %>
+
+<% button_bar do %>
+  <%= button :back, _('Back to submissions'), :action => 'submissions', :id => @form.id %>
+<% end %>
@@ -0,0 +1,25 @@
+<% self.extend CustomFormsPlugin::Helper %>
+
+<h1><%= _('Submissions for %s') % @form.name %></h1>
+
+<% if @form.submissions.empty? %>
+  <%= _('There are no submissions for this form.') %>
+<% else %>
+  <table class="action-table">
+    <tr>
+      <th style='width: 50%'><%= _('Author') %></th>
+      <th style='width: 50%'><%= _('Time') %></th>
+    </tr>
+    <% @submissions.each do |submission| %>
+      <tr>
+        <% author = submission.profile.present? ? submission.profile.name : submission.author_name %>
+        <td><%= link_to(author, {:action => 'show_submission', :id => submission.id}) %></td>
+        <td><%= time_format(submission.created_at) %></td>
+      </tr>
+    <% end %>
+  </table>
+<% end %>
+
+<% button_bar do %>
+  <%= button :back, _('Back to forms'), :action => 'index' %>
+<% end %>
@@ -0,0 +1,23 @@
+<h1><%= @form.name %></h1>
+<p><%= @form.description %></p>
+
+<% if @submission.id.nil? %>
+  <%= error_messages_for :submission %>
+
+  <% form_for :submission, @submission do |f| %>
+    <% if !user %>
+      <%= required labelled_form_field _('Author name'), text_field_tag(:author_name, @submission.author_name) %>
+      <%= required labelled_form_field _('Author email'), text_field_tag(:author_email, @submission.author_email) %>
+    <% end %>
+
+    <%= render :partial => 'shared/form_submission', :locals => {:f => f} %>
+
+    <% button_bar do %>
+      <%= submit_button :save, _('Save'), :cancel => {:action => 'index'} %>
+    <% end %>
+  <% end %>
+<% else %>
+  <% fields_for :submission, @submission do |f| %>
+    <%= render :partial => 'shared/form_submission', :locals => {:f => f} %>
+  <% end %>
+<% end %>
@@ -0,0 +1,5 @@
+<% self.extend(CustomFormsPlugin::Helper) %>
+
+<% @form.fields.each do |field| %>
+  <%= display_custom_field(field, @submission, f.object_name) %>
+<% end %>
@@ -0,0 +1 @@
+<%= f.object_name.to_s %>
@@ -0,0 +1,10 @@
+<% @form = CustomFormsPlugin::Form.find(task.form_id) %>
+<% @submission = CustomFormsPlugin::Submission.new(:form_id => @form.id, :profile_id => user.id) %>
+
+<h2><%= @form.name %></h2>
+<p><%= @form.description %></p>
+
+<% f.fields_for :submission do |fi| %>
+  <%#= fi.error_messages_for :submission %>
+  <%= render :partial => 'shared/form_submission', :locals => {:f => fi} %>
+<% end %>
@@ -0,0 +1,3 @@
+class FooPluginAdminBarController < AdminController
+end
+
@@ -0,0 +1,3 @@
+class FooPluginMyprofileBarController < MyProfileController
+end
+
@@ -0,0 +1,3 @@
+class FooPluginProfileBarController < ProfileController
+end
+
@@ -0,0 +1,2 @@
+class FooPluginPublicBarController < PublicController
+end
@@ -2,7 +2,15 @@ class MezuroPluginMyprofileController &lt; ProfileController
   append_view_path File.join(File.dirname(__FILE__) + '/../views')
- 
+  rescue_from Exception do |exception|
+    message = URI.escape(CGI.escape(exception.message),'.')
+    redirect_to_error_page message
+  end
+
+  def error_page
+    @message = params[:message]
+  end
+
   def choose_base_tool
     @configuration_content = profile.articles.find(params[:id])
     @base_tools = Kalibro::BaseTool.all_names
@@ -11,19 +19,23 @@ class MezuroPluginMyprofileController &lt; ProfileController
   def choose_metric
     @configuration_content = profile.articles.find(params[:id])
     @base_tool = params[:base_tool]
-    @supported_metrics = Kalibro::BaseTool.find_by_name(@base_tool).supported_metrics
+    base_tool = Kalibro::BaseTool.find_by_name(@base_tool)
+    @supported_metrics = base_tool.nil? ? [] : base_tool.supported_metrics 
   end
-  
+
   def new_metric_configuration
     @configuration_content = profile.articles.find(params[:id])
     @metric = Kalibro::BaseTool.find_by_name(params[:base_tool]).metric params[:metric_name]
   end
-  
+
   def new_compound_metric_configuration
     @configuration_content = profile.articles.find(params[:id])
     @metric_configurations = @configuration_content.metric_configurations
+    if configuration_content_has_errors?
+      redirect_to_error_page @configuration_content.errors[:base]
+    end
   end
-  
+
   def edit_metric_configuration
     @configuration_content = profile.articles.find(params[:id])
     @metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(@configuration_content.name, params[:metric_name])
@@ -36,19 +48,29 @@ class MezuroPluginMyprofileController &lt; ProfileController
     @metric_configurations = @configuration_content.metric_configurations
     @metric = @metric_configuration.metric
   end
-  
+
   def create_metric_configuration
     id = params[:id]
     metric_name = params[:metric_configuration][:metric][:name]
-    (Kalibro::MetricConfiguration.new(params[:metric_configuration])).save
-    redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/edit_metric_configuration?id=#{id}&metric_name=#{metric_name.gsub(/\s/, '+')}"
+    metric_configuration = Kalibro::MetricConfiguration.new(params[:metric_configuration])
+    metric_configuration.save
+    if metric_configuration_has_errors? metric_configuration
+      redirect_to_error_page metric_configuration.errors[0].message
+    else
+      redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/edit_metric_configuration?id=#{id}&metric_name=#{metric_name.gsub(/\s/, '+')}"
+    end
   end
-  
+
   def create_compound_metric_configuration
     id = params[:id]
     metric_name = params[:metric_configuration][:metric][:name]
-    Kalibro::MetricConfiguration.new(params[:metric_configuration]).save
-    redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/edit_compound_metric_configuration?id=#{id}&metric_name=#{metric_name.gsub(/\s/, '+')}"
+    metric_configuration = Kalibro::MetricConfiguration.new(params[:metric_configuration])
+    metric_configuration.save
+    if metric_configuration_has_errors? metric_configuration
+      redirect_to_error_page metric_configuration.errors[0].message
+    else
+      redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/edit_compound_metric_configuration?id=#{id}&metric_name=#{metric_name.gsub(/\s/, '+')}"
+    end
   end
   def update_metric_configuration
@@ -56,7 +78,11 @@ class MezuroPluginMyprofileController &lt; ProfileController
     metric_name = params[:metric_configuration][:metric][:name]
     metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(@configuration_content.name, metric_name)
     metric_configuration.update_attributes params[:metric_configuration]
-    redirect_to "/#{profile.identifier}/#{@configuration_content.slug}"
+    if metric_configuration_has_errors? metric_configuration
+      redirect_to_error_page metric_configuration.errors[0].message
+    else
+      redirect_to "/#{profile.identifier}/#{@configuration_content.slug}"
+    end
   end
   def update_compound_metric_configuration
@@ -64,7 +90,11 @@ class MezuroPluginMyprofileController &lt; ProfileController
     metric_name = params[:metric_configuration][:metric][:name]
     metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(@configuration_content.name, metric_name)
     metric_configuration.update_attributes params[:metric_configuration]
-    redirect_to "/#{profile.identifier}/#{@configuration_content.slug}"
+    if metric_configuration_has_errors? metric_configuration
+      redirect_to_error_page metric_configuration.errors[0].message
+    else
+      redirect_to "/#{profile.identifier}/#{@configuration_content.slug}"
+    end
   end
   def remove_metric_configuration
@@ -72,32 +102,41 @@ class MezuroPluginMyprofileController &lt; ProfileController
     metric_name = params[:metric_name]
     metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(configuration_content.name, metric_name)
     metric_configuration.destroy
-    redirect_to "/#{profile.identifier}/#{configuration_content.slug}"
+    if metric_configuration_has_errors? metric_configuration
+      redirect_to_error_page metric_configuration.errors[0].message
+    else
+      redirect_to "/#{profile.identifier}/#{configuration_content.slug}"
+    end
   end
-  
+
   def new_range
     @configuration_content = profile.articles.find(params[:id])
     @metric_name = params[:metric_name]
     @range = Kalibro::Range.new
+    @range_color = "#000000"
   end
-  
+
   def edit_range
     @configuration_content = profile.articles.find(params[:id])
     @metric_name = params[:metric_name]
     @beginning_id = params[:beginning_id]
     metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(@configuration_content.name, @metric_name)
     @range = metric_configuration.ranges.find{|range| range.beginning == @beginning_id.to_f || @beginning_id =="-INF" }
+    @range_color = "#" + @range.color.to_s.gsub(/^ff/, "")
   end
   def create_range
     @configuration_content = profile.articles.find(params[:id])
     @range = Kalibro::Range.new params[:range]
     metric_name = params[:metric_name]
-    metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(@configuration_content.name, metric_name)   
+    metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(@configuration_content.name, metric_name)
     metric_configuration.add_range(@range)
     metric_configuration.save
+    if metric_configuration_has_errors? metric_configuration
+      redirect_to_error_page metric_configuration.errors[0].message
+    end
   end
-  
+
   def update_range
     configuration_content = profile.articles.find(params[:id])
     metric_name = params[:metric_name]
@@ -106,8 +145,11 @@ class MezuroPluginMyprofileController &lt; ProfileController
     index = metric_configuration.ranges.index{ |range| range.beginning == beginning_id.to_f || beginning_id == "-INF" }
     metric_configuration.ranges[index] = Kalibro::Range.new params[:range]
     metric_configuration.save
+    if metric_configuration_has_errors? metric_configuration
+      redirect_to_error_page metric_configuration.errors[0].message
+    end
   end
-  
+
   def remove_range
     configuration_content = profile.articles.find(params[:id])
     metric_name = params[:metric_name]
@@ -115,12 +157,31 @@ class MezuroPluginMyprofileController &lt; ProfileController
     metric_configuration = Kalibro::MetricConfiguration.find_by_configuration_name_and_metric_name(configuration_content.name, metric_name)
     metric_configuration.ranges.delete_if { |range| range.beginning == beginning_id.to_f || beginning_id == "-INF" }
     metric_configuration.save
-    formatted_metric_name = metric_name.gsub(/\s/, '+')
-    if metric_configuration.metric.class == Kalibro::CompoundMetric
-      redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/edit_compound_metric_configuration?id=#{configuration_content.id}&metric_name=#{formatted_metric_name}"
+    if metric_configuration_has_errors? metric_configuration
+      redirect_to_error_page metric_configuration.errors[0].message
     else
-      redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/edit_metric_configuration?id=#{configuration_content.id}&metric_name=#{formatted_metric_name}"
+      formatted_metric_name = metric_name.gsub(/\s/, '+')
+      if metric_configuration.metric.class == Kalibro::CompoundMetric
+        redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/edit_compound_metric_configuration?id=#{configuration_content.id}&metric_name=#{formatted_metric_name}"
+      else
+        redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/edit_metric_configuration?id=#{configuration_content.id}&metric_name=#{formatted_metric_name}"
+      end
     end
   end
+  private
+
+  def redirect_to_error_page(message)
+    message = URI.escape(CGI.escape(message),'.')
+    redirect_to "/myprofile/#{profile.identifier}/plugin/mezuro/error_page?message=#{message}"
+  end
+
+  def configuration_content_has_errors?
+    not @configuration_content.errors[:base].nil?
+  end
+
+  def metric_configuration_has_errors? metric_configuration
+    not metric_configuration.errors.empty?
+  end
+
 end
 class MezuroPluginProfileController < ProfileController
   append_view_path File.join(File.dirname(__FILE__) + '/../views')
-  
+
+  def error_page
+    @message = params[:message]
+  end
+
   def project_state
     @content = profile.articles.find(params[:id])
     project = @content.project
-    state = project.error.nil? ? project.state : "ERROR"
-    render :text => state
+    if project_content_has_errors?
+      redirect_to_error_page(@content.errors[:base])
+    else
+      state = project.kalibro_error.nil? ? project.state : "ERROR"
+      render :text => state
+    end
   end
   def project_error
     @content = profile.articles.find(params[:id])
     @project = @content.project
-    render :partial => 'content_viewer/project_error'
+    if project_content_has_errors?
+      redirect_to_error_page(@content.errors[:base])
+    else
+      render :partial => 'content_viewer/project_error'
+    end
   end
   def project_result
     @content = profile.articles.find(params[:id])
     date = params[:date]
     @project_result = date.nil? ? @content.project_result : @content.project_result_with_date(date)
-    render :partial => 'content_viewer/project_result'
-  end 	
+    if project_content_has_errors?
+      redirect_to_error_page(@content.errors[:base])
+    else
+      render :partial => 'content_viewer/project_result'
+    end
+  end
   def module_result
     @content = profile.articles.find(params[:id])
     @module_result = @content.module_result(params)
-    render :partial => 'content_viewer/module_result'
+    @module = @module_result.module
+    @module_label = "#{@module.name} (#{@module.granularity})"
+    if project_content_has_errors?
+      redirect_to_error_page(@content.errors[:base])
+    else
+      render :partial => 'content_viewer/module_result'
+    end
   end
   def project_tree
     @content = profile.articles.find(params[:id])
     date = params[:date]
     project_result = date.nil? ? @content.project_result : @content.project_result_with_date(date)
-    @project_name = @content.project.name
-    @source_tree = project_result.node_of(params[:module_name])
-    render :partial =>'content_viewer/source_tree'
+    @project_name = @content.project.name if not @content.project.nil?
+    if project_content_has_errors?
+      redirect_to_error_page(@content.errors[:base])
+    else
+      @source_tree = project_result.node(params[:module_name])
+      render :partial =>'content_viewer/source_tree'
+    end
   end
   def module_metrics_history
     metric_name = params[:metric_name]
     @content = profile.articles.find(params[:id])
     module_history = @content.result_history(params[:module_name])
-    @score_history = filtering_metric_history(metric_name, module_history)
-    render :partial => 'content_viewer/score_history'
+    if project_content_has_errors?
+      redirect_to_error_page(@content.errors[:base])
+    else
+      @score_history = filtering_metric_history(metric_name, module_history)
+      render :partial => 'content_viewer/score_history'
+    end
   end
   def module_grade_history
     @content = profile.articles.find(params[:id])
     modules_results = @content.result_history(params[:module_name])
-    @score_history = modules_results.collect { |module_result| module_result.grade }
-    render :partial => 'content_viewer/score_history'
+    if project_content_has_errors?
+      redirect_to_error_page(@content.errors[:base])
+    else
+      @score_history = modules_results.map do |module_result|
+        [module_result.grade, format_date_to_simple_form(module_result.date)]
+      end
+      render :partial => 'content_viewer/score_history'
+    end
   end
-  
+
   private
-  
+
   def filtering_metric_history(metric_name, module_history)
     metrics_history = module_history.map do |module_result|
-      module_result.metric_results
+      [module_result.metric_results, format_date_to_simple_form(module_result.date)]
     end
-    metric_history =  metrics_history.map do |array_of_metric_result|
-      (array_of_metric_result.select do |metric_result|
+    metric_history =  metrics_history.map do |metric_results_with_date|
+      [(metric_results_with_date.first.select do |metric_result|
         metric_result.metric.name.delete("() ") == metric_name
-      end).first
+      end).first, metric_results_with_date.last]
     end
-    metric_history.map do |metric_result|
-      metric_result.value
+    metric_history.map do |metric_result_with_date|
+      [metric_result_with_date.first.value, metric_result_with_date.last]
     end
   end
+
+  def redirect_to_error_page(message)
+    message = URI.escape(CGI.escape(message),'.')
+    redirect_to "/profile/#{profile.identifier}/plugins/mezuro/error_page?message=#{message}"
+  end
+
+  def project_content_has_errors?
+    not @content.errors[:base].nil?
+  end
+
+  def format_date_to_simple_form date
+    date.to_s[0..9]
+  end
+
 end
@@ -7,11 +7,7 @@ class Kalibro::Configuration &lt; Kalibro::Model
   end
   def metric_configurations
-    if @metric_configuration != nil
-      @metric_configuration
-    else
-      []
-    end
+    @metric_configuration.nil? ? [] : @metric_configuration
   end
   def metric_configurations=(metric_configurations)
@@ -19,19 +15,11 @@ class Kalibro::Configuration &lt; Kalibro::Model
   end
   def self.find_by_name(configuration_name)
-    begin
-      new request("Configuration", :get_configuration, {:configuration_name => configuration_name})[:configuration]
-    rescue Exception => error
-      nil
-    end
+    new request("Configuration", :get_configuration, {:configuration_name => configuration_name})[:configuration]
   end
   def self.all_names
-    begin
-      request("Configuration", :get_configuration_names)[:configuration_name]
-    rescue Exception
-      []
-    end
+    request("Configuration", :get_configuration_names)[:configuration_name]
   end
   def update_attributes(attributes={})
@@ -49,10 +49,14 @@ class Kalibro::MetricConfiguration &lt; Kalibro::Model
   end
   def destroy
-    self.class.request("MetricConfiguration", :remove_metric_configuration, {
+    begin
+      self.class.request("MetricConfiguration", :remove_metric_configuration, {
         :configuration_name => configuration_name,
         :metric_name=> metric.name
       })
+    rescue Exception => exception
+      add_error exception
+    end
   end
   def to_hash
 class Kalibro::Model
+  attr_accessor :errors
+
   def initialize(attributes={})
     attributes.each { |field, value| send("#{field}=", value) if self.class.is_valid?(field) }
+    @errors = []
   end
   def to_hash(options={})
     hash = Hash.new
     excepts = !options[:except].nil? ? options[:except] : []
+    excepts << :errors
     fields.each do |field|
       if(!excepts.include?(field))
         field_value = send(field)
@@ -46,15 +50,17 @@ class Kalibro::Model
     begin
       self.class.request(save_endpoint, save_action, save_params)
 	    true
-	  rescue Exception => error
-		  false
+	  rescue Exception => exception
+	    add_error exception
+	    false
 	  end
   end
   def destroy
     begin
       self.class.request(destroy_endpoint, destroy_action, destroy_params)
-    rescue Exception
+    rescue Exception => exception
+	    add_error exception
     end
   end
@@ -123,4 +129,9 @@ class Kalibro::Model
     {"#{class_name.underscore}_name".to_sym => self.name}
   end
+  def add_error(exception)
+    @errors << exception
+  end
+
 end
+
 class Kalibro::Project < Kalibro::Model
-  attr_accessor :name, :license, :description, :repository, :configuration_name, :state, :error
+  attr_accessor :name, :license, :description, :repository, :configuration_name, :state, :kalibro_error
   def self.all_names
     request("Project", :get_project_names)[:project_name]
@@ -15,23 +15,35 @@ class Kalibro::Project &lt; Kalibro::Model
   end
   def error=(value)
-    @error = Kalibro::Error.to_object value
+    @kalibro_error = Kalibro::Error.to_object value
   end
   def process_project(days = '0')
-    if days.to_i.zero?
-      self.class.request("Kalibro", :process_project, {:project_name => name})
-  	else
-      self.class.request("Kalibro", :process_periodically, {:project_name => name, :period_in_days => days})
-  	end
+    begin
+      if days.to_i.zero?
+        self.class.request("Kalibro", :process_project, {:project_name => name})
+    	else
+        self.class.request("Kalibro", :process_periodically, {:project_name => name, :period_in_days => days})
+    	end
+    rescue Exception => exception
+      add_error exception
+    end
   end
 	def process_period
-    self.class.request("Kalibro", :get_process_period, {:project_name => name})[:period]
+	  begin
+      self.class.request("Kalibro", :get_process_period, {:project_name => name})[:period]
+    rescue Exception => exception
+      add_error exception
+    end
   end
 	def cancel_periodic_process
-    self.class.request("Kalibro", :cancel_periodic_process, {:project_name => name})
+	  begin
+      self.class.request("Kalibro", :cancel_periodic_process, {:project_name => name})
+    rescue Exception => exception
+      add_error exception
+    end
   end
 end
@@ -75,21 +75,17 @@ class Kalibro::ProjectResult &lt; Kalibro::Model
     ('%2d' % amount).sub(/\s/, '0')
   end
-  def node_of(module_name)
+  def node(module_name)
     if module_name.nil? or module_name == project.name
       node = source_tree
     else
-      node = get_node(module_name)
-    end
-  end
-
-  def get_node(module_name)
-    path = Kalibro::Module.parent_names(module_name)
-    parent = @source_tree
-    path.each do |node_name|
-      parent = get_leaf_from(parent, node_name)
-    end
-    return parent
+		path = Kalibro::Module.parent_names(module_name)
+		parent = @source_tree
+		path.each do |node_name|
+		  parent = get_leaf_from(parent, node_name)
+		end
+		parent
+	end
   end
   private
@@ -34,4 +34,12 @@ class Kalibro::Range &lt; Kalibro::Model
     @grade = value.to_f
   end
+	def mezuro_color
+		@color.nil? ? "#e4ca2d" : @color.gsub(/^ff/, "#")
+	end
+	
+	def color=(new_color)
+		@color = new_color.gsub(/^#/, "ff")
+	end
+	
 end
@@ -3,8 +3,8 @@ class MezuroPlugin::ConfigurationContent &lt; Article
   settings_items :description, :configuration_to_clone_name
-  after_save :send_configuration_to_service
-  after_destroy :remove_configuration_from_service
+  after_save :send_kalibro_configuration_to_service
+  after_destroy :remove_kalibro_configuration_from_service
   def self.short_description
     'Kalibro configuration'
@@ -21,54 +21,60 @@ class MezuroPlugin::ConfigurationContent &lt; Article
     end
   end
-  def configuration
-    @configuration ||= Kalibro::Configuration.find_by_name(self.name)
-    if @configuration.nil? 
-      errors.add_to_base("Kalibro Configuration not found")
+  def kalibro_configuration
+    begin
+      @kalibro_configuration ||= Kalibro::Configuration.find_by_name(self.name)
+    rescue Exception => exception 
+      errors.add_to_base(exception.message)
     end
-    @configuration
+    @kalibro_configuration
   end
   def metric_configurations
-    configuration.metric_configurations
+    kalibro_configuration.metric_configurations
   end
-  def configuration_names
-    ["None"] + Kalibro::Configuration.all_names.sort
+  def kalibro_configuration_names
+    begin
+      ["None"] + Kalibro::Configuration.all_names.sort
+    rescue Exception => exception
+      errors.add_to_base(exception.message)
+      ["None"]
+    end
   end
   private
   def validate_kalibro_configuration_name
-    existing = configuration_names.map { |a| a.downcase}
+    existing = kalibro_configuration_names.map { |a| a.downcase}
     if existing.include?(name.downcase)
       errors.add_to_base("Configuration name already exists in Kalibro")
     end
   end
-  def send_configuration_to_service
-    if editing_configuration?
-      configuration.update_attributes({:description => description})
+  def send_kalibro_configuration_to_service
+    if editing_kalibro_configuration?
+      kalibro_configuration.update_attributes({:description => description})
     else
       create_kalibro_configuration
     end
   end
-  def remove_configuration_from_service
-    configuration.destroy
+  def remove_kalibro_configuration_from_service
+    kalibro_configuration.destroy unless kalibro_configuration.nil?
   end
   def create_kalibro_configuration
     attributes = {:name => name, :description => description}
-    if cloning_configuration?
+    if cloning_kalibro_configuration?
       attributes[:metric_configuration] = configuration_to_clone.metric_configurations_hash
     end
     Kalibro::Configuration.create attributes
   end
-  def editing_configuration?
-    configuration.present?
+  def editing_kalibro_configuration?
+    kalibro_configuration.present?
   end
   def configuration_to_clone
@@ -76,10 +82,10 @@ class MezuroPlugin::ConfigurationContent &lt; Article
   end
   def find_configuration_to_clone
-    configuration_to_clone_name.nil? ? nil : Kalibro::Configuration.find_by_name(configuration_to_clone_name)
+    (configuration_to_clone_name == "None") ? nil : Kalibro::Configuration.find_by_name(configuration_to_clone_name)
   end
-  def cloning_configuration?
+  def cloning_kalibro_configuration?
     configuration_to_clone.present?
   end
 class MezuroPlugin::Helpers::ContentViewerHelper
+
+  MAX_NUMBER_OF_LABELS = 5
+
   def self.format_grade(grade)
     sprintf("%.2f", grade.to_f)
   end
@@ -6,15 +9,31 @@ class MezuroPlugin::Helpers::ContentViewerHelper
   def self.create_periodicity_options
    [["Not Periodically", 0], ["1 day", 1], ["2 days", 2], ["Weekly", 7], ["Biweeky", 15], ["Monthly", 30]]
   end
-  
-  def self.generate_chart(values)
-    Gchart.line(  
+
+  def self.create_license_options
+   options = YAML.load_file("#{RAILS_ROOT}/plugins/mezuro/licenses.yaml")
+   options = options.split(";")
+   formated_options = []
+   options.each { |option| formated_options << [option, option] }
+   formated_options
+  end
+
+  def self.generate_chart(score_history)
+    values = []
+    labels = []
+    score_history.each do |score_data|
+      values << score_data.first
+      labels << score_data.last
+    end
+    labels = discretize_array labels
+    Gchart.line(
                 :title_color => 'FF0000',
                 :size => '600x180', 
                 :bg => {:color => 'efefef', :type => 'stripes'},
                 :line_colors => 'c4a000',
                 :data => values,
-                :axis_with_labels => 'y',
+                :labels => labels,
+                :axis_with_labels => ['y','x'],
                 :max_value => values.max,
                 :min_value => values.min
                 )
@@ -25,8 +44,33 @@ class MezuroPlugin::Helpers::ContentViewerHelper
     selected_option = options.find { |option| option.last == index.to_i }
     selected_option.first
   end
-  
+
   def self.format_name(metric_result)
     metric_result.metric.name.delete("() ")
   end
+
+  def self.get_license_option(selected)
+    options = YAML.load_file("#{RAILS_ROOT}/plugins/mezuro/licenses.yaml")
+    options.split(";")
+    selected_option = options.find { |license| license == selected }
+  end
+
+  private
+
+  def self.discretize_array(array)
+    if array.size > MAX_NUMBER_OF_LABELS
+      range_array.map { |i| discrete_element(array, i)}
+    else
+      array
+    end
+  end
+
+  def self.range_array
+    (0..(MAX_NUMBER_OF_LABELS - 1)).to_a
+  end
+
+  def self.discrete_element(array, i)
+    array[(i*(array.size - 1))/(MAX_NUMBER_OF_LABELS - 1)]
+  end
+
 end
 class MezuroPlugin::ProjectContent < Article
   include ActionView::Helpers::TagHelper
-  settings_items :license, :description, :repository_type, :repository_url, :configuration_name, :periodicity_in_days
+  settings_items :project_license, :description, :repository_type, :repository_url, :configuration_name, :periodicity_in_days
   validate_on_create :validate_kalibro_project_name 
   validate_on_create :validate_repository_url
@@ -26,6 +26,7 @@ class MezuroPlugin::ProjectContent &lt; Article
     rescue Exception => error
       errors.add_to_base(error.message)
     end
+    @project
   end
   def project_result
@@ -34,6 +35,7 @@ class MezuroPlugin::ProjectContent &lt; Article
     rescue Exception => error
       errors.add_to_base(error.message)
     end
+    @project_result
   end
   def project_result_with_date(date)
@@ -43,6 +45,7 @@ Kalibro::ProjectResult.first_result_after(name, date)
     rescue Exception => error
       errors.add_to_base(error.message)
     end
+    @project_result
   end
   def module_result(attributes)
@@ -53,6 +56,7 @@ Kalibro::ProjectResult.first_result_after(name, date)
     rescue Exception => error
       errors.add_to_base(error.message)
     end
+    @module_result
   end
   def result_history(module_name)
@@ -73,6 +77,7 @@ Kalibro::ProjectResult.first_result_after(name, date)
       existing = Kalibro::Project.all_names
     rescue Exception => error
       errors.add_to_base(error.message)
+      existing = []
     end
     if existing.any?{|existing_name| existing_name.casecmp(name)==0} # existing.include?(name) + case insensitive
@@ -94,7 +99,7 @@ Kalibro::ProjectResult.first_result_after(name, date)
   def create_kalibro_project
    Kalibro::Project.create(
       :name => name,
-      :license => license,
+      :license => project_license,
       :description => description,
       :repository => {
         :type => repository_type,
@@ -105,7 +110,7 @@ Kalibro::ProjectResult.first_result_after(name, date)
   end
   def destroy_project_from_service
-    project.destroy
+    project.destroy unless project.nil?
   end
 end
@@ -0,0 +1,69 @@
+Academic Free License 3.0 (AFL-3.0);
+Affero GNU Public License (AGPL-3.0);
+Adaptive Public License (APL-1.0);
+Apache License 2.0 (Apache-2.0);
+Apple Public Source License (APSL-2.0);
+Artistic license 2.0 (Artistic-2.0);
+Attribution Assurance Licenses (AAL);
+BSD 3-Clause "New" or "Revised" License (BSD-3-Clause);
+BSD 2-Clause "Simplified" or "FreeBSD" License (BSD-2-Clause);
+Boost Software License (BSL-1.0);
+Computer Associates Trusted Open Source License 1.1 (CATOSL-1.1);
+Common Development and Distribution License 1.0 (CDDL-1.0);
+Common Public Attribution License 1.0 (CPAL-1.0);
+CUA Office Public License Version 1.0 (CUA-OPL-1.0);
+EU DataGrid Software License (EUDatagrid);
+Eclipse Public License 1.0 (EPL-1.0);
+Educational Community License, Version 2.0 (ECL-2.0);
+Eiffel Forum License V2.0 (EFL-2.0);
+Entessa Public License (Entessa);
+European Union Public License, Version 1.1 (EUPL-1.1);
+Fair License (FAIR);
+Frameworx License (Frameworx-1.0);
+GNU Affero General Public License v3 (AGPL-3.0);
+GNU General Public License version 2.0 (GPL-2.0);
+GNU General Public License version 3.0 (GPL-3.0);
+GNU Library or "Lesser" General Public License version 2.1 (LGPL-2.1);
+GNU Library or "Lesser" General Public License version 3.0 (LGPL-3.0);
+Historical Permission Notice and Disclaimer (HPND);
+IBM Public License 1.0 (IPL-1.0);
+IPA Font License (IPA);
+ISC License (ISC);
+LaTeX Project Public License 1.3c (LPPL-1.3c);
+Lucent Public License Version 1.02 (LPL-1.02);
+MirOS Licence (MirOS);
+Microsoft Public License (Ms-PL);
+Microsoft Reciprocal License (Ms-RL);
+MIT license (MIT);
+Motosoto License (Motosoto);
+Mozilla Public License 2.0 (MPL-2.0);
+Multics License (Multics);
+NASA Open Source Agreement 1.3 (NASA 1.3);
+NTP License (NTP);
+Naumen Public License (Naumen);
+Nethack General Public License (NGPL);
+Nokia Open Source License (Nokia);
+Non-Profit Open Software License 3.0 (NPOSL-3.0);
+OCLC Research Public License 2.0 (OCLC-2.0);
+Open Font License 1.1 (OFL 1.1);
+Open Group Test Suite License (OGTSL);
+Open Software License 3.0 (OSL-3.0);
+PHP License 3.0 (PHP-3.0);
+The PostgreSQL License (PostgreSQL);
+Python License (Python-2.0);
+CNRI Python license (CNRI-Python);
+Q Public License (QPL-1.0);
+RealNetworks Public Source License V1.0 (RPSL-1.0);
+Reciprocal Public License 1.5 (RPL-1.5);
+Ricoh Source Code Public License (RSCPL);
+Simple Public License 2.0 (SimPL-2.0);
+Sleepycat License (Sleepycat);
+Sun Public License 1.0 (SPL-1.0);
+Sybase Open Watcom Public License 1.0 (Watcom-1.0);
+University of Illinois/NCSA Open Source License (NCSA);
+Vovida Software License v. 1.0 (VSL-1.0);
+W3C License (W3C);
+wxWindows Library License (WXwindows);
+X.Net License (Xnet);
+Zope Public License 2.0 (ZPL-2.0);
+zlib/libpng license (Zlib);
@@ -0,0 +1,31 @@
+div.colorPicker-picker {
+  height: 16px;
+  width: 16px;
+  padding: 0 !important;
+  border: 1px solid #ccc;
+  background: url(arrow.gif) no-repeat top right;
+  cursor: pointer;
+  line-height: 16px;
+}
+
+div.colorPicker-palette {
+  width: 110px;
+  position: absolute;
+  border: 1px solid #598FEF;
+  background-color: #EFEFEF;
+  padding: 2px;
+  z-index: 9999;
+}
+  div.colorPicker_hexWrap {width: 100%; float:left }
+  div.colorPicker_hexWrap label {font-size: 95%; color: #2F2F2F; margin: 5px 2px; width: 25%}
+  div.colorPicker_hexWrap input {margin: 5px 2px; padding: 0; font-size: 95%; border: 1px solid #000; width: 65%; }
+
+div.colorPicker-swatch {
+  height: 12px;
+  width: 12px;
+  border: 1px solid #000;
+  margin: 2px;
+  float: left;
+  cursor: pointer;
+  line-height: 12px;
+}
@@ -0,0 +1,22 @@
+Copyright (c) 2012 Lakshan Perera
+
+Permission is hereby granted, free of charge, to any person
+obtaining a copy of this software and associated documentation
+files (the "Software"), to deal in the Software without
+restriction, including without limitation the rights to use,
+copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the
+Software is furnished to do so, subject to the following
+conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+OTHER DEALINGS IN THE SOFTWARE.
	@@ -5,7 +5,7 @@ class ProfileDesignController < BoxOrganizerController		@@ -5,7 +5,7 @@ class ProfileDesignController < BoxOrganizerController
5	protect 'edit_profile_design', :profile	5	protect 'edit_profile_design', :profile
6		6
7	def available_blocks	7	def available_blocks
8	- blocks = [ ArticleBlock, TagsBlock, RecentDocumentsBlock, ProfileInfoBlock, LinkListBlock, MyNetworkBlock, FeedReaderBlock, ProfileImageBlock, LocationBlock, SlideshowBlock, ProfileSearchBlock ]	8	+ blocks = [ ArticleBlock, TagsBlock, RecentDocumentsBlock, ProfileInfoBlock, LinkListBlock, MyNetworkBlock, FeedReaderBlock, ProfileImageBlock, LocationBlock, SlideshowBlock, ProfileSearchBlock, HighlightsBlock ]
9		9
10	# blocks exclusive for organizations	10	# blocks exclusive for organizations
11	if profile.has_members?	11	if profile.has_members?
	@@ -2,6 +2,8 @@ class ContentViewerController < ApplicationController		@@ -2,6 +2,8 @@ class ContentViewerController < ApplicationController
2		2
3	needs_profile	3	needs_profile
4		4
		5	+ before_filter :comment_author, :only => :edit_comment
		6	+
5	helper ProfileHelper	7	helper ProfileHelper
6	helper TagsHelper	8	helper TagsHelper
7		9
	@@ -121,6 +123,27 @@ class ContentViewerController < ApplicationController		@@ -121,6 +123,27 @@ class ContentViewerController < ApplicationController
121	end	123	end
122	end	124	end
123		125
		126	+ def edit_comment
		127	+ path = params[:page].join('/')
		128	+ @page = profile.articles.find_by_path(path)
		129	+ @form_div = 'opened'
		130	+ @comment = @page.comments.find_by_id(params[:id])
		131	+ if @comment
		132	+ if request.post?
		133	+ begin
		134	+ @comment.update_attributes(params[:comment])
		135	+ session[:notice] = _('Comment succesfully updated')
		136	+ redirect_to :action => 'view_page', :profile => profile.identifier, :page => @comment.article.explode_path
		137	+ rescue
		138	+ session[:notice] = _('Comment could not be updated')
		139	+ end
		140	+ end
		141	+ else
		142	+ redirect_to @page.view_url
		143	+ session[:notice] = _('Could not find the comment in the article')
		144	+ end
		145	+ end
		146	+
124	protected	147	protected
125		148
126	def add_comment	149	def add_comment
	@@ -198,4 +221,13 @@ class ContentViewerController < ApplicationController		@@ -198,4 +221,13 @@ class ContentViewerController < ApplicationController
198	end	221	end
199	end	222	end
200		223
		224	+ def comment_author
		225	+ comment = Comment.find_by_id(params[:id])
		226	+ if comment
		227	+ render_access_denied if comment.author.blank? \|\| comment.author != user
		228	+ else
		229	+ render_not_found
		230	+ end
		231	+ end
		232	+
201	end	233	end
	@@ -155,6 +155,119 @@ module FormsHelper		@@ -155,6 +155,119 @@ module FormsHelper
155	return result	155	return result
156	end	156	end
157		157
		158	+ def date_field(name, value, format = '%Y-%m-%d', datepicker_options = {}, html_options = {})
		159	+ datepicker_options[:disabled] \|\|= false
		160	+ datepicker_options[:alt_field] \|\|= ''
		161	+ datepicker_options[:alt_format] \|\|= ''
		162	+ datepicker_options[:append_text] \|\|= ''
		163	+ datepicker_options[:auto_size] \|\|= false
		164	+ datepicker_options[:button_image] \|\|= ''
		165	+ datepicker_options[:button_image_only] \|\|= false
		166	+ datepicker_options[:button_text] \|\|= '...'
		167	+ datepicker_options[:calculate_week] \|\|= 'jQuery.datepicker.iso8601Week'
		168	+ datepicker_options[:change_month] \|\|= false
		169	+ datepicker_options[:change_year] \|\|= false
		170	+ datepicker_options[:close_text] \|\|= _('Done')
		171	+ datepicker_options[:constrain_input] \|\|= true
		172	+ datepicker_options[:current_text] \|\|= _('Today')
		173	+ datepicker_options[:date_format] \|\|= 'mm/dd/yy'
		174	+ datepicker_options[:day_names] \|\|= [_('Sunday'), _('Monday'), _('Tuesday'), _('Wednesday'), _('Thursday'), _('Friday'), _('Saturday')]
		175	+ datepicker_options[:day_names_min] \|\|= [_('Su'), _('Mo'), _('Tu'), _('We'), _('Th'), _('Fr'), _('Sa')]
		176	+ datepicker_options[:day_names_short] \|\|= [_('Sun'), _('Mon'), _('Tue'), _('Wed'), _('Thu'), _('Fri'), _('Sat')]
		177	+ datepicker_options[:default_date] \|\|= nil
		178	+ datepicker_options[:duration] \|\|= 'normal'
		179	+ datepicker_options[:first_day] \|\|= 0
		180	+ datepicker_options[:goto_current] \|\|= false
		181	+ datepicker_options[:hide_if_no_prev_next] \|\|= false
		182	+ datepicker_options[:is_rtl] \|\|= false
		183	+ datepicker_options[:max_date] \|\|= nil
		184	+ datepicker_options[:min_date] \|\|= nil
		185	+ datepicker_options[:month_names] \|\|= [_('January'), _('February'), _('March'), _('April'), _('May'), _('June'), _('July'), _('August'), _('September'), _('October'), _('November'), _('December')]
		186	+ datepicker_options[:month_names_short] \|\|= [_('Jan'), _('Feb'), _('Mar'), _('Apr'), _('May'), _('Jun'), _('Jul'), _('Aug'), _('Sep'), _('Oct'), _('Nov'), _('Dec')]
		187	+ datepicker_options[:navigation_as_date_format] \|\|= false
		188	+ datepicker_options[:next_text] \|\|= _('Next')
		189	+ datepicker_options[:number_of_months] \|\|= 1
		190	+ datepicker_options[:prev_text] \|\|= _('Prev')
		191	+ datepicker_options[:select_other_months] \|\|= false
		192	+ datepicker_options[:short_year_cutoff] \|\|= '+10'
		193	+ datepicker_options[:show_button_panel] \|\|= false
		194	+ datepicker_options[:show_current_at_pos] \|\|= 0
		195	+ datepicker_options[:show_month_after_year] \|\|= false
		196	+ datepicker_options[:show_on] \|\|= 'focus'
		197	+ datepicker_options[:show_options] \|\|= {}
		198	+ datepicker_options[:show_other_months] \|\|= false
		199	+ datepicker_options[:show_week] \|\|= false
		200	+ datepicker_options[:step_months] \|\|= 1
		201	+ datepicker_options[:week_header] \|\|= _('Wk')
		202	+ datepicker_options[:year_range] \|\|= 'c-10:c+10'
		203	+ datepicker_options[:year_suffix] \|\|= ''
		204	+
		205	+ element_id = html_options[:id] \|\| 'datepicker-date'
		206	+ value = value.strftime(format) if value.present?
		207	+ method = datepicker_options[:time] ? 'datetimepicker' : 'datepicker'
		208	+ result = text_field_tag(name, value, html_options)
		209	+ result +=
		210	+ "
		211	+ <script type='text/javascript'>
		212	+ jQuery('##{element_id}').#{method}({
		213	+ disabled: #{datepicker_options[:disabled].to_json},
		214	+ altField: #{datepicker_options[:alt_field].to_json},
		215	+ altFormat: #{datepicker_options[:alt_format].to_json},
		216	+ appendText: #{datepicker_options[:append_text].to_json},
		217	+ autoSize: #{datepicker_options[:auto_size].to_json},
		218	+ buttonImage: #{datepicker_options[:button_image].to_json},
		219	+ buttonImageOnly: #{datepicker_options[:button_image_only].to_json},
		220	+ buttonText: #{datepicker_options[:button_text].to_json},
		221	+ calculateWeek: #{datepicker_options[:calculate_week].to_json},
		222	+ changeMonth: #{datepicker_options[:change_month].to_json},
		223	+ changeYear: #{datepicker_options[:change_year].to_json},
		224	+ closeText: #{datepicker_options[:close_text].to_json},
		225	+ constrainInput: #{datepicker_options[:constrain_input].to_json},
		226	+ currentText: #{datepicker_options[:current_text].to_json},
		227	+ dateFormat: #{datepicker_options[:date_format].to_json},
		228	+ dayNames: #{datepicker_options[:day_names].to_json},
		229	+ dayNamesMin: #{datepicker_options[:day_names_min].to_json},
		230	+ dayNamesShort: #{datepicker_options[:day_names_short].to_json},
		231	+ defaultDate: #{datepicker_options[:default_date].to_json},
		232	+ duration: #{datepicker_options[:duration].to_json},
		233	+ firstDay: #{datepicker_options[:first_day].to_json},
		234	+ gotoCurrent: #{datepicker_options[:goto_current].to_json},
		235	+ hideIfNoPrevNext: #{datepicker_options[:hide_if_no_prev_next].to_json},
		236	+ isRTL: #{datepicker_options[:is_rtl].to_json},
		237	+ maxDate: #{datepicker_options[:max_date].to_json},
		238	+ minDate: #{datepicker_options[:min_date].to_json},
		239	+ monthNames: #{datepicker_options[:month_names].to_json},
		240	+ monthNamesShort: #{datepicker_options[:month_names_short].to_json},
		241	+ navigationAsDateFormat: #{datepicker_options[:navigation_as_date_format].to_json},
		242	+ nextText: #{datepicker_options[:next_text].to_json},
		243	+ numberOfMonths: #{datepicker_options[:number_of_months].to_json},
		244	+ prevText: #{datepicker_options[:prev_text].to_json},
		245	+ selectOtherMonths: #{datepicker_options[:select_other_months].to_json},
		246	+ shortYearCutoff: #{datepicker_options[:short_year_cutoff].to_json},
		247	+ showButtonPanel: #{datepicker_options[:show_button_panel].to_json},
		248	+ showCurrentAtPos: #{datepicker_options[:show_current_at_pos].to_json},
		249	+ showMonthAfterYear: #{datepicker_options[:show_month_after_year].to_json},
		250	+ showOn: #{datepicker_options[:show_on].to_json},
		251	+ showOptions: #{datepicker_options[:show_options].to_json},
		252	+ showOtherMonths: #{datepicker_options[:show_other_months].to_json},
		253	+ showWeek: #{datepicker_options[:show_week].to_json},
		254	+ stepMonths: #{datepicker_options[:step_months].to_json},
		255	+ weekHeader: #{datepicker_options[:week_header].to_json},
		256	+ yearRange: #{datepicker_options[:year_range].to_json},
		257	+ yearSuffix: #{datepicker_options[:year_suffix].to_json}
		258	+ })
		259	+ </script>
		260	+ "
		261	+ result
		262	+ end
		263	+
		264	+ def date_range_field(from_name, to_name, from_value, to_value, format = '%Y-%m-%d', datepicker_options = {}, html_options = {})
		265	+ from_id = html_options[:from_id] \|\| 'datepicker-from-date'
		266	+ to_id = html_options[:to_id] \|\| 'datepicker-to-date'
		267	+ return _('From') +' '+ date_field(from_name, from_value, format, datepicker_options, html_options.merge({:id => from_id})) +
		268	+ ' ' + _('until') +' '+ date_field(to_name, to_value, format, datepicker_options, html_options.merge({:id => to_id}))
		269	+ end
		270	+
158	protected	271	protected
159	def self.next_id_number	272	def self.next_id_number
160	if defined? @@id_num	273	if defined? @@id_num
	@@ -2,4 +2,76 @@ class Box < ActiveRecord::Base		@@ -2,4 +2,76 @@ class Box < ActiveRecord::Base
2	belongs_to :owner, :polymorphic => true	2	belongs_to :owner, :polymorphic => true
3	acts_as_list :scope => 'owner_id = #{owner_id} and owner_type = \'#{owner_type}\''	3	acts_as_list :scope => 'owner_id = #{owner_id} and owner_type = \'#{owner_type}\''
4	has_many :blocks, :dependent => :destroy, :order => 'position'	4	has_many :blocks, :dependent => :destroy, :order => 'position'
		5	+
		6	+ def acceptable_blocks
		7	+ to_css_class_name central? ? Box.acceptable_center_blocks : Box.acceptable_side_blocks
		8	+ end
		9	+
		10	+ def central?
		11	+ position == 1
		12	+ end
		13	+
		14	+ def self.acceptable_center_blocks
		15	+ [ ArticleBlock,
		16	+ BlogArchivesBlock,
		17	+ CategoriesBlock,
		18	+ CommunitiesBlock,
		19	+ EnterprisesBlock,
		20	+ EnvironmentStatisticsBlock,
		21	+ FansBlock,
		22	+ FavoriteEnterprisesBlock,
		23	+ FeedReaderBlock,
		24	+ FriendsBlock,
		25	+ HighlightsBlock,
		26	+ LinkListBlock,
		27	+ LoginBlock,
		28	+ MainBlock,
		29	+ MembersBlock,
		30	+ MyNetworkBlock,
		31	+ PeopleBlock,
		32	+ ProfileImageBlock,
		33	+ RawHTMLBlock,
		34	+ RecentDocumentsBlock,
		35	+ SellersSearchBlock,
		36	+ TagsBlock ]
		37	+ end
		38	+
		39	+ def self.acceptable_side_blocks
		40	+ [ ArticleBlock,
		41	+ BlogArchivesBlock,
		42	+ CategoriesBlock,
		43	+ CommunitiesBlock,
		44	+ DisabledEnterpriseMessageBlock,
		45	+ EnterprisesBlock,
		46	+ EnvironmentStatisticsBlock,
		47	+ FansBlock,
		48	+ FavoriteEnterprisesBlock,
		49	+ FeaturedProductsBlock,
		50	+ FeedReaderBlock,
		51	+ FriendsBlock,
		52	+ HighlightsBlock,
		53	+ LinkListBlock,
		54	+ LocationBlock,
		55	+ LoginBlock,
		56	+ MembersBlock,
		57	+ MyNetworkBlock,
		58	+ PeopleBlock,
		59	+ ProductsBlock,
		60	+ ProfileImageBlock,
		61	+ ProfileInfoBlock,
		62	+ ProfileSearchBlock,
		63	+ RawHTMLBlock,
		64	+ RecentDocumentsBlock,
		65	+ SellersSearchBlock,
		66	+ SlideshowBlock,
		67	+ TagsBlock
		68	+ ]
		69	+ end
		70	+
		71	+ private
		72	+
		73	+ def to_css_class_name(blocks)
		74	+ blocks.map{ \|block\| block.to_s.underscore.tr('_', '-') }
		75	+ end
		76	+
5	end	77	end
	@@ -28,6 +28,8 @@ class Comment < ActiveRecord::Base		@@ -28,6 +28,8 @@ class Comment < ActiveRecord::Base
28		28
29	xss_terminate :only => [ :body, :title, :name ], :on => 'validation'	29	xss_terminate :only => [ :body, :title, :name ], :on => 'validation'
30		30
		31	+ delegate :environment, :to => :source
		32	+
31	def action_tracker_target	33	def action_tracker_target
32	self.article.profile	34	self.article.profile
33	end	35	end
	@@ -31,7 +31,7 @@ class Task < ActiveRecord::Base		@@ -31,7 +31,7 @@ class Task < ActiveRecord::Base
31	end	31	end
32	end	32	end
33		33
34	- belongs_to :requestor, :class_name => 'Person', :foreign_key => :requestor_id	34	+ belongs_to :requestor, :class_name => 'Profile', :foreign_key => :requestor_id
35	belongs_to :target, :foreign_key => :target_id, :polymorphic => true	35	belongs_to :target, :foreign_key => :target_id, :polymorphic => true
36		36
37	validates_uniqueness_of :code, :on => :create	37	validates_uniqueness_of :code, :on => :create
@@ -0,0 +1,10 @@		@@ -0,0 +1,10 @@
	1	+<% block_types.in_groups_of(2) do \|block1, block2\| %>
	2	+ <div style='float: left; width: 48%; padding-top: 2px;'>
	3	+ <%= labelled_radio_button(block1.description, :type, block1.name) %>
	4	+ </div>
	5	+ <% if block2 %>
	6	+ <div style='float: left; width: 48%; padding-top: 2px;'>
	7	+ <%= labelled_radio_button(block2.description, :type, block2.name) %>
	8	+ </div>
	9	+ <% end %>
	10	+<% end %>
1	<strong><%= _('Highlights') %></strong>	1	<strong><%= _('Highlights') %></strong>
2	-<div id='edit-highlights-block'>	2	+<div id='edit-highlights-block' style='width:450px'>
3	<table id='highlights' class='noborder'>	3	<table id='highlights' class='noborder'>
4	<tr><th><%= _('Image') %></th><th><%= _('Address') %></th><th><%= _('Position') %></th><th><%= _('Title') %></th></tr>	4	<tr><th><%= _('Image') %></th><th><%= _('Address') %></th><th><%= _('Position') %></th><th><%= _('Title') %></th></tr>
5	<% for image in @block.images do %>	5	<% for image in @block.images do %>
1	<strong><%= _('Links') %></strong>	1	<strong><%= _('Links') %></strong>
2	-<div id='edit-link-list-block'>	2	+<div id='edit-link-list-block' style='width:450px'>
3	<table id='links' class='noborder'>	3	<table id='links' class='noborder'>
4	<tr><th><%= _('Icon') %></th><th><%= _('Name') %></th><th><%= _('Address') %></th></tr>	4	<tr><th><%= _('Icon') %></th><th><%= _('Name') %></th><th><%= _('Address') %></th></tr>
5	<% for link in @block.links do %>	5	<% for link in @block.links do %>
1	-<% form_tag do %>
2	-
3	- <p><%= _('In what area do you want to put your new block?') %></p>
4	-
5	- <%# FIXME hardcoded stuff %>
6	- <%= select_tag('box_id', options_for_select(@boxes.select { \|item\| item.position != 1 }.map {\|item\| [ _("Area %d") % item.position, item.id]})) %>
7	-
8	- <p><%= _('Select the type of block you want to add to your page.') %></p>
9	-
10	- <% @block_types.in_groups_of(2) do \|block1, block2\| %>
11	- <div style='float: left; width: 48%; padding-top: 2px;'>
12	- <%= radio_button_tag('type', block1.name) %>
13	- <%= label_tag "type_#{block1.name.downcase}", block1.description %>	1	+<div style='height:350px'>
		2	+ <% form_tag do %>
		3	+
		4	+ <p><%= _('In what area do you want to put your new block?') %></p>
		5	+
		6	+ <% @boxes.each do \|box\| %>
		7	+ <%= labelled_radio_button(_("Area %d") % box.position, :box_id, box.id, box.central?, { :class => 'box-position', 'data-position' => box.position }) %>
		8	+ <% end %>
		9	+
		10	+ <script type="text/javascript">
		11	+ (function ($) {
		12	+ $(document).ready(function () {
		13	+ $(".box-position").live('change', function () {
		14	+ if ($(this).attr('data-position') == '1') {
		15	+ $('#center-block-types').show();
		16	+ $('#side-block-types').hide();
		17	+ } else {
		18	+ $('#center-block-types').hide();
		19	+ $('#side-block-types').show();
		20	+ };
		21	+ });
		22	+ })})(jQuery);
		23	+ </script>
		24	+
		25	+ <p><%= _('Select the type of block you want to add to your page.') %></p>
		26	+
		27	+ <div id='center-block-types'>
		28	+ <%= render :partial => 'block_types', :locals => { :block_types => @center_block_types } %>
14	</div>	29	</div>
15	- <% if block2 %>
16	- <div style='float: left; width: 48%; padding-top: 2px;'>
17	- <%= radio_button_tag('type', block2.name) %>
18	- <%= label_tag "type_#{block2.name.downcase}", block2.description %>
19	- </div>	30	+
		31	+ <div id='side-block-types' style='display:none'>
		32	+ <%= render :partial => 'block_types', :locals => { :block_types => @side_block_types } %>
		33	+ </div>
		34	+
		35	+ <br style='clear: both'/>
		36	+
		37	+ <% button_bar do %>
		38	+ <%= submit_button(:add, _("Add")) %>
		39	+ <%= colorbox_close_button(_('Close')) %>
20	<% end %>	40	<% end %>
21	- <% end %>
22	- <br style='clear: both'/>
23	-
24	- <% button_bar do %>
25	- <%= submit_button(:add, _("Add")) %>
26	- <%= lightbox_close_button(_('Close')) %>
27	- <% end %>
28		41
29	-<% end %>	42	+ <% end %>
		43	+</div>