[Mezuro] Fixed color picker.

Diego Camarinha · Diego Camarinha
1 parent aca22dd2
Showing 5 changed files with 28 additions and 23 deletions Show diff stats
plugins/mezuro/controllers/mezuro_plugin_myprofile_controller.rb
plugins/mezuro/lib/kalibro/range.rb
plugins/mezuro/public/javascripts/validations.js
plugins/mezuro/test/unit/kalibro/range_test.rb
plugins/mezuro/views/mezuro_plugin_myprofile/_range_form.html.erb
@@ -113,7 +113,6 @@ class MezuroPluginMyprofileController &lt; ProfileController
     @configuration_content = profile.articles.find(params[:id])
     @metric_name = params[:metric_name]
     @range = Kalibro::Range.new
-    @range_color = "#000000"
   end
   def edit_range
@@ -122,7 +121,6 @@ class MezuroPluginMyprofileController &lt; ProfileController
     @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
@@ -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
@@ -39,14 +39,6 @@ function IsNotInfinite(value){
     return true;
 }
-function IsNotHexadecimal(value){
-    if(value.match(/^[0-9a-fA-F]{1,8}$/))
-    {
-      return false;
-    }
-    return true;
-}
-
 function validate_new_range_configuration(event){    
     var label = jQuery("#range_label").val();
     var beginning = jQuery("#range_beginning").val();
@@ -56,24 +48,20 @@ function validate_new_range_configuration(event){
     if (is_null(label) || is_null(beginning) || is_null(end) || is_null(color) || is_null(grade))
     {
-        alert("Please fill all fields marked with (*)");
+        alert("Please fill all fields marked with (*).");
         return false;
     }
     if ( (IsNotNumeric(beginning) && IsNotInfinite(beginning)) || (IsNotNumeric(end) && IsNotInfinite(end)) || IsNotNumeric(grade))
     {
-        alert("Beginning, End and Grade must be numeric values");
+        alert("Beginning, End and Grade must be numeric values.");
         return false;
     }
     if (parseInt(beginning) > parseInt(end))
     {
         if(IsNotInfinite(beginning) && IsNotInfinite(end)){
-          alert("End must be greater than Beginning");
+          alert("End must be greater than Beginning.");
           return false;
         }
     }
-    if (IsNotHexadecimal(color)){
-        alert("Color must be an hexadecimal value");
-        return false;
-    }
     return true;
 }
@@ -17,4 +17,16 @@ class RangeTest &lt; ActiveSupport::TestCase
     assert_equal @hash, @range.to_hash
   end
+	should 'create a default color for new range' do 
+		assert_equal "#e4ca2d", Kalibro::Range.new.mezuro_color
+	end
+	
+	should "convert color from 'ff' to '#'" do
+		assert_equal "#ff0000", @range.mezuro_color
+	end
+
+	should "convert color from '#' to 'ff' when creating a new range" do
+		assert_equal "ffff0000", Kalibro::Range.new({:color => '#ff0000'}).color
+	end
+	
 end
@@ -38,7 +38,7 @@
       <%= f.label :color, "(*) Color:" %>
     </td>
     <td>
-      <%= f.text_field(:color, :id => "range_color", :value => @range_color) %>
+      <%= f.text_field(:color, :id => "range_color", :value => @range.mezuro_color) %>
     </td>
   </tr>
   <tr>
@@ -53,10 +53,9 @@
 <%= f.submit "Save Range" %>
 <script>jQuery(document).ready(function($) {
-   $('#range_color').colorPicker({ 
-   	onColorChange : function(id, newValue) {
-   		newValue = newValue.replace(/#/gi, "ff");
-   		jQuery('#range_color').val(newValue);
-   	}
+    $('#range_color').colorPicker({ 
+   		onColorChange : function(id, newValue) {
+   			jQuery('#range_color').val(newValue);
+   		}
    	});
 });</script>
	@@ -34,4 +34,12 @@ class Kalibro::Range < Kalibro::Model		@@ -34,4 +34,12 @@ class Kalibro::Range < Kalibro::Model
34	@grade = value.to_f	34	@grade = value.to_f
35	end	35	end
36		36
		37	+ def mezuro_color
		38	+ @color.nil? ? "#e4ca2d" : @color.gsub(/^ff/, "#")
		39	+ end
		40	+
		41	+ def color=(new_color)
		42	+ @color = new_color.gsub(/^#/, "ff")
		43	+ end
		44	+
37	end	45	end
	@@ -17,4 +17,16 @@ class RangeTest < ActiveSupport::TestCase		@@ -17,4 +17,16 @@ class RangeTest < ActiveSupport::TestCase
17	assert_equal @hash, @range.to_hash	17	assert_equal @hash, @range.to_hash
18	end	18	end
19		19
		20	+ should 'create a default color for new range' do
		21	+ assert_equal "#e4ca2d", Kalibro::Range.new.mezuro_color
		22	+ end
		23	+
		24	+ should "convert color from 'ff' to '#'" do
		25	+ assert_equal "#ff0000", @range.mezuro_color
		26	+ end
		27	+
		28	+ should "convert color from '#' to 'ff' when creating a new range" do
		29	+ assert_equal "ffff0000", Kalibro::Range.new({:color => '#ff0000'}).color
		30	+ end
		31	+
20	end	32	end