Resolve [Ruby help] Script for create window sills

[benj59380]

Hello , Today I wanted Creat script for window sills

my approach is create face manuel
click for select

run script
( part 1 : select face , creat color , Push -30 mm
Part 2 : select vertical face for push +30)

Part 1 work , but Part 2 fail

But Ruby = SyntaxError help me please for resolve the problem
thank you in advance , Benjamin

#part 1
model = Sketchup.active_model
ent = model.entities
sel = model.selection
ad = sel.ent

faces = []

sel.each do |e|
faces <<e if e.is_a? Sketchup::Face
end

faces.each do |appui|

appuicolor = Sketchup::Color.new(238,130,238)
appui.material = appuicolor
appui.back_material = appuicolor

status = appui.pushpull(-30.mm , true)

end
#part 2

ad.each do |e|

s=Sketchup.active_model.selection;
a=s.to_a;s.clear;
a.grep(Sketchup::Face).each{|f|s.add(f)if f.normal.z==0}

faces.each do |facade|

status = facade.pushpull(30.mm )

end

[sdmitch]

Part 1 works if you delete or comment out the ad=sel.ent statement which is invalid.

Part 2 doesn't work because the faces created by the pushpull operation are not automatically added to the selection. Also you reuse the faces array which only contains the original faces selected.

Here is how I would do it

mod = Sketchup.active_model
ent = mod.active_entities
sel = mod.selection
SKETCHUP_CONSOLE.clear
appuicolor = Sketchup;;Color.new(238,130,238)
sel.grep(Sketchup;;Face).each do |appui|
  appui.material = appuicolor
  appui.back_material = appuicolor
  old = ent.to_a
  status = appui.pushpull(-30.mm , true)
  new = ent.to_a - old
  new.grep(Sketchup;;Face).select{|f|f.normal.z==0}.each{|f|f.pushpull 30.mm}
end

[benj59380]

hello , Thank you SDMITCH for help

cordially Benj