search

thomthom / solid-inspector

Inspect and repair solid groups and components in SketchUp.
https://extensions.sketchup.com/en/content/solid-inspector%C2%B2
MIT License
18 stars 2 forks source link

Extensions interfere with JSON module #41

Open thomthom opened 8 years ago

thomthom commented 8 years ago

Original report by me.

SU Walk:

After installing:

String.instance_method(:to_json).source_location
["c:/programdata/sketchup/sketchup 2016/sketchup/plugins/su_walk/plug_in/sketchup_json.rbs", 264]

Logged error report:

http://sketchup.thomthom.net/extensions/TT_SolidInspector2/reports/report/1441

#<SketchUpJSON::JSONEncodeError: This hash can not generate valid JSON>

c:/programdata/sketchup/sketchup 2016/sketchup/plugins/su_walk/plug_in/sketchup_json.rbs:316:in `validate_key!'
c:/programdata/sketchup/sketchup 2016/sketchup/plugins/su_walk/plug_in/sketchup_json.rbs:307:in `block in to_json'
c:/programdata/sketchup/sketchup 2016/sketchup/plugins/su_walk/plug_in/sketchup_json.rbs:306:in `each'
c:/programdata/sketchup/sketchup 2016/sketchup/plugins/su_walk/plug_in/sketchup_json.rbs:306:in `collect'
c:/programdata/sketchup/sketchup 2016/sketchup/plugins/su_walk/plug_in/sketchup_json.rbs:306:in `to_json'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/window.rb:60:in `block in call'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/window.rb:60:in `map'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/window.rb:60:in `call'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:590:in `update_webdialog'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:257:in `analyze'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:403:in `block in create_window'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/window.rb:95:in `call'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/window.rb:95:in `block in trigger_events'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/window.rb:94:in `each'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/window.rb:94:in `trigger_events'
C:/Users/Lichen Intern/AppData/Roaming/SketchUp/SketchUp 2016/SketchUp/Plugins/tt_solid_inspector2/window.rb:35:in `block in initialize'
SketchUp:1:in `call'
thomthom commented 8 years ago

HelioScope

http://www.folsomlabs.com/

http://sketchup.thomthom.net/extensions/TT_SolidInspector2/reports/report/1391?page=1

#<HelioScope::JSON::JSONEncodeError: This hash can not generate valid JSON>

c:/users/secret engineering/appdata/roaming/sketchup/sketchup 2015/sketchup/plugins/helioscope_ext/json/generator.rbs:72:in `validate_key!'
c:/users/secret engineering/appdata/roaming/sketchup/sketchup 2015/sketchup/plugins/helioscope_ext/json/generator.rbs:63:in `block in to_json'
c:/users/secret engineering/appdata/roaming/sketchup/sketchup 2015/sketchup/plugins/helioscope_ext/json/generator.rbs:62:in `each'
c:/users/secret engineering/appdata/roaming/sketchup/sketchup 2015/sketchup/plugins/helioscope_ext/json/generator.rbs:62:in `collect'
c:/users/secret engineering/appdata/roaming/sketchup/sketchup 2015/sketchup/plugins/helioscope_ext/json/generator.rbs:62:in `to_json'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:60:in `block in call'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:60:in `map'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:60:in `call'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:590:in `update_webdialog'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:257:in `analyze'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:498:in `fix_all'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:407:in `block in create_window'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:95:in `call'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:95:in `block in trigger_events'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:94:in `each'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:94:in `trigger_events'
C:/Users/Secret Engineering/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:35:in `block in initialize'
SketchUp:1:in `call'
thomthom commented 8 years ago

Bim Bon

http://sketchup.thomthom.net/extensions/TT_SolidInspector2/reports/report/1624

#<SketchUpJSON::JSONEncodeError: This hash can not generate valid JSON>

C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/bimbon/lib/sketchup_json/generator.rb:71:in `validate_key!'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/bimbon/lib/sketchup_json/generator.rb:62:in `block in to_json'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/bimbon/lib/sketchup_json/generator.rb:61:in `each'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/bimbon/lib/sketchup_json/generator.rb:61:in `collect'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/bimbon/lib/sketchup_json/generator.rb:61:in `to_json'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:60:in `block in call'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:60:in `map'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:60:in `call'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:590:in `update_webdialog'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:257:in `analyze'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:498:in `fix_all'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/inspector_tool.rb:407:in `block in create_window'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:95:in `call'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:95:in `block in trigger_events'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:94:in `each'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:94:in `trigger_events'
C:/Users/Obras/AppData/Roaming/SketchUp/SketchUp 2015/SketchUp/Plugins/tt_solid_inspector2/window.rb:35:in `block in initialize'
SketchUp:1:in `call'

Unable to reproduce with latest version (1.1.2) on EW. Error might be from older version?

thomthom commented 8 years ago

Hi Thom,

SU Walk is actually a stand alone application for the most part. But it has a couple of plulg-ins for SketchUp that help users create "fake" reflections (mirrors) and bumps on materials that SU Walk can read from the imported .skp file.

This plug-ins were written original for SketchUp 7 and it has not changed much. At the time the developer (Taka) of SU Walk felt _jsons method I would guess. I will ask him. Perhaps the stdlib did not have what he needed.

The only thing we have done to keep this compatible with current releases of SU Walk is to update the skp reader.

This program is hardly used and I'm surprised that anyone has had conflicts with it. I'm not sure if we will continue selling it after SU 2017. Most of our attention is on SU Podium, Podium Browser, Podium Walker.

I'll talk to Taka and a couple of developers about switching the StdLib.

Dave