[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [25258] trunk/blender/source/blender/ editors: Patch [#20290] by Bj?\195?\184rnar Hansen for bug [#20270] 2.5a0, Select Random says percent but is really 0-1

[Martin Poirier]

Revision: 25258
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=25258
Author:   theeth
Date:     2009-12-09 22:42:21 +0100 (Wed, 09 Dec 2009)

Log Message:
-----------
Patch [#20290] by Bj?\195?\184rnar Hansen for bug [#20270] 2.5a0, Select Random says percent but is really 0-1

Modified Paths:
--------------
    trunk/blender/source/blender/editors/mesh/editmesh_mods.c
    trunk/blender/source/blender/editors/object/object_select.c

Modified: trunk/blender/source/blender/editors/mesh/editmesh_mods.c
===================================================================
--- trunk/blender/source/blender/editors/mesh/editmesh_mods.c	2009-12-09 20:03:08 UTC (rev 25257)
+++ trunk/blender/source/blender/editors/mesh/editmesh_mods.c	2009-12-09 21:42:21 UTC (rev 25258)
@@ -3629,7 +3629,7 @@
 	if(!RNA_boolean_get(op->ptr, "extend"))
 		EM_deselect_all(em);
 	
-	selectrandom_mesh(em, RNA_float_get(op->ptr, "percentage")/100.0f);
+	selectrandom_mesh(em, RNA_float_get(op->ptr, "percent")/100.0f);
 		
 	WM_event_add_notifier(C, NC_GEOM|ND_SELECT, obedit->data);
 	
@@ -3652,8 +3652,8 @@
 	ot->flag= OPTYPE_REGISTER|OPTYPE_UNDO;
 	
 	/* props */
-	RNA_def_float_percentage(ot->srna, "percentage", 50.f, 0.0f, 100.0f, "Percentage", "Percentage of elements to select randomly.", 0.f, 100.0f);
-	RNA_def_boolean(ot->srna, "extend", 0, "Extend Select", "");
+	RNA_def_float_percentage(ot->srna, "percent", 50.f, 0.0f, 100.0f, "Percent", "Percentage of elements to select randomly.", 0.f, 100.0f);
+	RNA_def_boolean(ot->srna, "extend", FALSE, "Extend Selection", "Extend selection instead of deselecting everything first.");
 }
 
 void EM_select_by_material(EditMesh *em, int index) 

Modified: trunk/blender/source/blender/editors/object/object_select.c
===================================================================
--- trunk/blender/source/blender/editors/object/object_select.c	2009-12-09 20:03:08 UTC (rev 25257)
+++ trunk/blender/source/blender/editors/object/object_select.c	2009-12-09 21:42:21 UTC (rev 25258)
@@ -996,11 +996,11 @@
 		}
 		CTX_DATA_END;
 	}
-	percent = RNA_float_get(op->ptr, "percent");
+	percent = RNA_float_get(op->ptr, "percent")/100.0f;
 		
 	CTX_DATA_BEGIN(C, Base*, base, visible_bases) {
 		if (BLI_frand() < percent) {
-				ED_base_object_select(base, BA_SELECT);
+			ED_base_object_select(base, BA_SELECT);
 		}
 	}
 	CTX_DATA_END;
@@ -1013,7 +1013,7 @@
 void OBJECT_OT_select_random(wmOperatorType *ot)
 {
 	/* identifiers */
-	ot->name= "Random select";
+	ot->name= "Select Random";
 	ot->description = "Set select on random visible objects.";
 	ot->idname= "OBJECT_OT_select_random";
 	
@@ -1026,8 +1026,8 @@
 	ot->flag= OPTYPE_REGISTER|OPTYPE_UNDO;
 	
 	/* properties */
-	RNA_def_boolean(ot->srna, "extend", FALSE, "Extend", "Extend selection instead of deselecting everything first.");
-	RNA_def_float_percentage(ot->srna, "percent", 0.5f, 0.0f, 1.0f, "Percent", "percentage of objects to randomly select", 0.0001f, 1.0f);
+	RNA_def_float_percentage(ot->srna, "percent", 50.f, 0.0f, 100.0f, "Percent", "Percentage of objects to select randomly", 0.f, 100.0f);
+	RNA_def_boolean(ot->srna, "extend", FALSE, "Extend Selection", "Extend selection instead of deselecting everything first.");
 }