How to add multiple groups into one group?

thomthom

It would help if you described what "doesn't work" - because running the code it doesn't produce any errors.

However, did you expect the component definition you added to be visible in the model?
If so then you must add an instance, using Entities.add_instance
http://www.sketchup.com/intl/en/developer/docs/ourdoc/entities.php#add_instance

thomthom

Other notes:

require 'Sketchup.rb'
The require is case sensitive - even though the file is the same if someone uses require 'sketchup.rb' and you later use require 'Sketchup.rb' then the file will be loaded twice.
Lower case is the norm for require 'sketchup.rb'.

And it's a strange pattern you have there - creating a class to execute a single operation. Normally you will create a class for for instance a Tool or when you have an object to represent - other a module is better suited.

It's also recommended not to use globals - as you might run into conflicts with other plugins.

More information here: http://www.thomthom.net/thoughts/2012/01/golden-rules-of-sketchup-plugin-development/

nithi09

I change to require 'sketchup.rb'. still nothing happening. It's suppose to draw a cube in sketcup model I don't see anything happening

thomthom

@nithi09 said:

I change to require 'sketchup.rb'. still nothing happening. It's suppose to draw a cube in sketcup model I don't see anything happening

See my first reply - about addin an instance of the definition you created.

(Also, when you modify the model you should wrap everything up in model.start_operation and model.commit_operation so it can be undone in one step.)

nithi09

I changed my code still not working here is the code after I changed

<span class="syntaxdefault"><br />require </span><span class="syntaxstring">'sketchup.rb'<br /><br /></span><span class="syntaxdefault">def cube<br />list </span><span class="syntaxkeyword">=</span><span class="syntaxdefault"> Sketchup</span><span class="syntaxkeyword">.</span><span class="syntaxdefault">active_model</span><span class="syntaxkeyword">.</span><span class="syntaxdefault">definitions<br />comp_def </span><span class="syntaxkeyword">=</span><span class="syntaxdefault"> list</span><span class="syntaxkeyword">.</span><span class="syntaxdefault">add </span><span class="syntaxstring">"CUBE BOX"<br /></span><span class="syntaxdefault">comp_def</span><span class="syntaxkeyword">.</span><span class="syntaxdefault">description </span><span class="syntaxkeyword">=</span><span class="syntaxdefault"> </span><span class="syntaxstring">"This is a simple cube component"<br /></span><span class="syntaxdefault">ents </span><span class="syntaxkeyword">=</span><span class="syntaxdefault"> comp_def</span><span class="syntaxkeyword">.</span><span class="syntaxdefault">entities</span><span class="syntaxkeyword">.</span><span class="syntaxdefault">add_instance<br />face </span><span class="syntaxkeyword">=</span><span class="syntaxdefault"> ents</span><span class="syntaxkeyword">.</span><span class="syntaxdefault">add_face </span><span class="syntaxkeyword">[</span><span class="syntaxdefault">0</span><span class="syntaxkeyword">,</span><span class="syntaxdefault">0</span><span class="syntaxkeyword">,</span><span class="syntaxdefault">0</span><span class="syntaxkeyword">],[</span><span class="syntaxdefault">24</span><span class="syntaxkeyword">,</span><span class="syntaxdefault">0</span><span class="syntaxkeyword">,</span><span class="syntaxdefault">0</span><span class="syntaxkeyword">],[</span><span class="syntaxdefault">24</span><span class="syntaxkeyword">,</span><span class="syntaxdefault">12</span><span class="syntaxkeyword">,</span><span class="syntaxdefault">0</span><span class="syntaxkeyword">],[</span><span class="syntaxdefault">0</span><span class="syntaxkeyword">,</span><span class="syntaxdefault">12</span><span class="syntaxkeyword">,</span><span class="syntaxdefault">0</span><span class="syntaxkeyword">]<br /></span><span class="syntaxdefault">face</span><span class="syntaxkeyword">.</span><span class="syntaxdefault">pushpull 16<br /><br />end<br /><br /><br />if </span><span class="syntaxkeyword">(</span><span class="syntaxdefault">not $Cube_menu_loaded</span><span class="syntaxkeyword">)<br /></span><span class="syntaxdefault">  <br />  UI</span><span class="syntaxkeyword">.</span><span class="syntaxdefault">menu</span><span class="syntaxkeyword">(</span><span class="syntaxstring">"Tools"</span><span class="syntaxkeyword">).</span><span class="syntaxdefault">add_item</span><span class="syntaxkeyword">(</span><span class="syntaxstring">"Cube_BOX"</span><span class="syntaxkeyword">)</span><span class="syntaxdefault"> </span><span class="syntaxkeyword">{</span><span class="syntaxdefault"> cube </span><span class="syntaxkeyword">}</span><span class="syntaxdefault"> <br />  <br />  $Cube_menu_loaded </span><span class="syntaxkeyword">=</span><span class="syntaxdefault"> true<br />end<br /></span>

TIG

Your code only does half the job.
It adds a 'box' [24x12x16"] to the model's definitions.
If you want to add an instance of that definition to the model you need to do more steps...

For example...
defn=list[-1] to get the last added definition
then to add an instance setup:
tr=Geom::Tranfsormation.new(ORIGIN)
OR other transformations like different points or rotations etc.
and use
instance=Sketchup.active_model.active_entities.add_instance(defn, tr)
NOW you have one instance of the cube component place at the 'ORIGIN' [0,0,0] or anywhere else you specify in 'tr'...

nithi09

this is different question.I want add color when OnClick. I try like this

group.set_attribute('dynamic_attributes', 'onclick',"Set("Material","Red","Blue","Green")")

but gives an error How do I do this
thanks

TIG

The best way to ensure you correctly 'phrase' it is to add the working onclick attribute manually to a DC [let's say it's reference is 'group'], then use:
att = group.get_attribute('dynamic_attributes', 'onclick', "???")
This will return att as ??? if it's not set, OR att as the attribute string, and this is in a form that you know works.
Then reuse that in your 'set'
group.set_attribute('dynamic_attributes', 'onclick', att) to see if the syntax is right...

BUT note this: even if the format is right, the way you are using nested " quote-marks WILL cause failures... USING
**"**Set("Material","Red","Blue","Green")**"**
WILL CAUSE AN ERROR - unless you either 'escape' the internal " with a , OR more simply you use a pair of ' to enclose the whole string...
i.e.
"Set(**\**"Material**\**",**\**"Red**\**",**\**"Blue**\**",**\**"Green**\**")"
or
**'**Set("Material","Red","Blue","Green")**'**
You must use a pair of ' to enclose ", or a pair of " to enclose any ' in a string... OR 'escape' the nested quote-marks as " or '

thomthom

@nithi09 said:

but gives an error How do I do this

It really helps if you include the error message when posting questions related to errors.

nithi09

Thanks TIG it's working when I use 'Set("Material","Red","Blue","Green")' but only problem I have to manually change to default color then onClick working fine How do I set to default color.

jolran

material = nil ?

nithi09

Thanks Jolran It's working now

nithi09

Is it possible to change ItemCode according to the onclick color change if it is how do I can do?

nithi09

Hope this explains my requirement a little better.

I have a sunroom comprised of many groups: corner post, floor channel, door, window etc.

I want the user to select their sunroom color by clicking the model. In order to do this I need to gather all the sunroom groups into a parent group where I will use the onClick function to change the Material attribute. I expect the Material change will automatically cascade to the sub-groups but if not I can easily set the Material of the sub-groups to something like=Parent!Material

The question is how do I wrap all of my groups into this parent group?

Thom Thom thank you for the advice regarding start opertaion, but I do not need undo for this model, the user is starting with a blank canvas and will simply delete top level group if they have to start over