[Plugin] Components manager plugin

inteloide · 14 Jan 2014, 22:54

Hello,

As every day, just post an update, with following corrections :
Corrected : When clicking on component in model, the information were not displayed.
Corrected : Revision system now working
Corrected : Bug when clicking on a group in the model (doesn't block user)
Corrected : Attempt to solve Mac file issue
Added : Spanish language, thanks to Oxer

JQL, this solve points 2 and 3. For your point 1, note that Sketchup force the file to be a ".skp" when saving component...No way to solve this...
About point 4 : it could be complex...I will try (I stack this with the filters for the list ;o)

Oxer : thank yoiu for translation !
About your issue on Mac, I make a correction that may solve your issue. I saw that the "/" and "" were different on your message. as windows reconize both, I force the string to be with "/". I guess it solve the problem...Please keep me informed !

Note also that I release the first help file (at least !!!). Hope you will love it !!!

bye !

Oxer · 15 Jan 2014, 02:04

Thanks for the update but it appears the same message in Ruby Console:

%(#FF0000)[Error al cargar archivo /Users/Oxer/Library/Application Support/SketchUp 2013/SketchUp/Plugins/BGSketchup/Composants/Rb/Composants_Main.rb
Permission denied - /Users/Oxer\BGSketchup_Composants.ini
Error al cargar archivo BGSketchup_Composants.rb
Permission denied - /Users/Oxer\BGSketchup_Composants.ini]

inteloide · 15 Jan 2014, 06:41

@oxer said:

Thanks for the update but it appears the same message in Ruby Console:

%(#FF0000)[Error al cargar archivo /Users/Oxer/Library/Application Support/SketchUp 2013/SketchUp/Plugins/BGSketchup/Composants/Rb/Composants_Main.rb
Permission denied - /Users/Oxer\BGSketchup_Composants.ini
Error al cargar archivo BGSketchup_Composants.rb
Permission denied - /Users/Oxer\BGSketchup_Composants.ini]

Hello Oxer,

could you please post the complete error message ? To which line produce this bug ?
thank you !

TIG · 15 Jan 2014, 10:35

The answer is in the text
/Users/Oxer****\****BGSketchup_Composants.ini
The backslash \ will only work as a file separator character on PCs, NOT MACs !
A / works on both platforms.
You should use File.join(directorypath, filename) to avoid this issue.
Also you can use .tr!("\\","/") to fix it.
[Conversely, on PC if you ever want to pass a path to say a "cmd" use .tr!("/","\\") as DOS always expects \ !]

On PC AND on MAC this will work:
/Users/Oxer****/****BGSketchup_Composants.ini

Change the code from:
file_name=BGSketchup_my_documents_dir()+"\\BGSketchup_Composants.ini"
to
file_name=Fiile.join(BGSketchup_my_documents_dir(), "BGSketchup_Composants.ini")
which avoids the \...

Incidentally don't set def methods with uppercase first letter:
BGSketchup_my_documents_dir() NO
bgSketchup_my_documents_dir() YES
Although a method named ' getTempFolder()' would be more logical*** ??

***Writing this ini file to that user-location is also somewhat ill-advised ?
Why not put it into a 'Temp folder' - this varies between PC and MAC...
ENV['TEMP'] || ENV['TMPDIR']
if !(mac) ... return path.strip.gsub("\000","") else #On Mac return File.expand_path "~" end
could simply be:
return ENV['TEMP'] || ENV['TMPDIR']
On PC it gives:
C:\Users\UserName\AppData\Local\Temp
OR perhaps on PC [ruby:3kd2xg63]ENV['APPDATA'][/ruby:3kd2xg63] giving:
[ruby:3kd2xg63]C:\Users\UserName\AppData\Roaming[/ruby:3kd2xg63]
On MAC it's the user's hidden ..../T[emp] folder
Keeping it 'out of the way', and avoiding all of the '.so' rigmarole and odd locations for the .ini file...

Oxer · 15 Jan 2014, 15:54

How TIG says, the answer is in the text, all red text is the Error Message.
Thanks TIG for the explanation, i don't know the ruby code.

inteloide · 15 Jan 2014, 19:17

Hello,

Thank you TIG for your help.
Your code is so much simple than mine...I changed to yours.

New version (1.9.4) solve only this error.

inteloide · 15 Jan 2014, 22:00

Version 1.9.5 :
Corrected : When saving all definitions, it asks now to save also file without link.
Optimzed : Now, for revision management, file with letter after revision level and accepted "toto rev 3 (super").skp"

Oxer, any feedback as a MAC user ?

I will officially post this version tomorrow if no bugs are found.

In parrallel I will look for JQL request but won't post new version (I think too much version make confusion...)

Oxer · 16 Jan 2014, 13:40

Oxer · 16 Jan 2014, 13:45

Now, there isn't Console Error when you open Sketchup, the settings window appears, the plugin generates the BGSketchup_Composants.ini file in Temporal folder, the translation works but the there are some problems:

The list is empty, when you push Update List button, nothing happens, no error in Ruby Console.
If you push Reload All Components button, it appears an error in Ruby Console, see the image 2.
If you push Save All Components button, it appears a first pop up window (image 3), i push Yes button and it appears a second empty pop up window (image 4), i push yes and it appears a Ruby Console Error (image 5).

TIG · 16 Jan 2014, 14:26

That ' nil' error points to code at line #494 of the script Composants_Main.rb - which reads:
UI.messagebox @strNbEnregistementDefinitions+" ("+compteur_ok.to_s+"/"+compteur.to_s+")"
So presumably a part of it is actually ' nil' ?
@strNbEnregistementDefinitions is set elsewhere in the script Localization.rb.
This is set from a series of ' if/elsif' tests, BUT ill-advisedly there is no 'fall-back' ' else' when nothing matches the other tests - so then it can return ' nil', and thereby cause the error - because a ' nil' object can't be used as part of a text string!

Perhaps change the code to read...
@strNbEnregistementDefinitions='?' unless @strNbEnregistementDefinitions UI.messagebox(@strNbEnregistementDefinitions+" ("+compteur_ok.to_s+"/"+compteur.to_s+")")

OR have a proper ' else' fall-back in the Localization.rb tests section...

The other two methods used in compiling the remainder of the string should always safely return a string as they start off as 0 - giving '... (1/2)' etc...

inteloide · 16 Jan 2014, 18:13

@tig said:

This is set from a series of ' if/elsif' tests, BUT ill-advisedly there is no 'fall-back' ' else' when nothing matches the other tests - so then it can return ' nil', and thereby cause the error - because a ' nil' object can't be used as part of a text string!

This could be due to a missing tranlsation in string (a string not defined in one language). Therefore, I change the way of working on Localisation.rb :
1-I load all strings in English
2-I load the strings in the wanted language.
This means that if a string is not translated, the string will be defined with English value.

The key thing, for me, is to always have strings defined in English, at minimal.

inteloide · 16 Jan 2014, 19:16

@oxer said:

If you push Reload All Components button, it appears an error in Ruby Console, see the image 2.

Please try version 1.9.6 below, which should solve this issue.

@oxer said:

The list is empty, when you push Update List button, nothing happens, no error in Ruby Console.

That's strange...maybe a javascript error...let's try with version 1.9.6, it may solve the issue.

Version 1.9.6

Oxer · 16 Jan 2014, 19:30

Ok, i'll test it.

Oxer · 16 Jan 2014, 20:20

The list continues empty
I'll test the other functions

Oxer · 16 Jan 2014, 20:32

Modify file's path -->works
Save selected component -->works
Save all components -->works
Save with new revision level -->works
Reload selected component -- > doesn't work, when you select the component it appears "Saved file is newer!" message but when i push Reload selected component button nothing happens, see the image below.

inteloide · 16 Jan 2014, 20:57

@oxer said:

The list continues empty
I'll test the other functions

Could be due to the browser I use which is IE.
Please try to open attached file by placing it in plugins folder, in HTML subfolder. And tells me what happen.
It's just a simplifyed list to see any error that your brownser may show.

If you see something like a list, try to open directly the treeview.html file.

Please report me any error...because without list...it's just not so useful...

I maybe need an html/javascript specialist...

The test file

Oxer · 16 Jan 2014, 21:21

I placed the test.html file in /Users/Oxer/Library/Application Support/SketchUp 2013/SketchUp/Plugins/BGSketchup/Composants/Html/ then I run Sketchup but when I call the plugin, the list is empty again. I have tried open tree.html in Safari browser and it appears a message:

Safari can't open "sip:get_data@charger_tires" because OSX doesn't recognize the address of internet that begin with "skp:".

When i open test.html in Safari the file shows like this:

inteloide · 18 Jan 2014, 10:28

Hello,

Just post new version, with following updates :
1.9.8: Corrected : Delete all debugging messages, now works on MAC ! Thanks to Oxer !

1.9.7:	Corrected :	Some javascript standard respect
	Added :		Show a message to update the list if the wasn't.
		Added :		Option not to start component mangement on SU start up to avoid adding attribute to model or slow down due to Observers

1.9.6:	Corrected :	For localisation, in cas of error, the English language is chosen, also, english tranlsation always preload in case translation is missing
	Corrected :	Change function in BGSketchup.rb functions into unic name function to avoid problem with other plugins

inteloide · 18 Jan 2014, 22:21

New version update (v 1.10.0) :
Added : Toolbar with icon to show the plugin window
Added : Help file is now in plugin sub folders, with a link from the "about" window. It was updated with last options explainations
Optimzed : "About" window in HTML format for better rendering and to allow links to help file and forum.
Optimzed : Component list shows scrollbar only is required.
Optimzed : On MAC, use plugin window is now always on top.
Corrected : Only one instance of the window can be shown.
Corrected : Added, some missing translations in Spanish language.

inteloide · 25 Jan 2014, 16:20

Hello,

Some updates today !

JQL, one of your request is complete (filters) !

1.11.0:
Added : New option to show list without a tree-style
Added : New button and window to filter item shown on list in case of non tree-style list
Corrected : Component with relative path were shown as "not updated", now shown in grey
Corrected : Tiny bug when showing components without dates
Corrected : Add filter in remove observer to avoid errors
Corrected : When clicking on component in model, all instances were selected if option Auto-select component in model was chosen
Optimzed : Purge only the definitions reloaded, to avoid any trouble with other definition

1.10.0:

Added : Toolbar with icon to show the plugin window
Added : Help file is now in plugin sub folders, with a link from the "about" window. It was updated with last options explainations
Optimzed : "About" window in HTML format for better rendering and to allow links to help file and forum.
Optimzed : Component list shows scrollbar only is required.
Optimzed : On MAC, use plugin window is now always on top.
Corrected : Only one instance of the window can be shown.
Corrected : Added, some missing translations in Spanish language.

I hope lots of you feel this plugins useful...
3D printer progress...only 1 euro by now...