1274 SDB.SelectFolder [#6056]

Beta Testing for Windows Products and plugins

Moderator: Gurus

Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

1274 SDB.SelectFolder [#6056]

Post by Owyn »

Code snippet.

Code: Select all

 ' Open inifile and get last used directory
  Dim iniF
  Set iniF = SDB.IniFile

  ' Let user select the output path
  Dim path
  path = iniF.StringValue( "Scripts", "LastDB_AuditDir")

  path = SDB.SelectFolder( path, SDB.Localize( "Select where to create the Audit Report."))

  If path="" Then
    Exit Sub
  End If

  If Right( path, 1)<>"\" Then
    path = path & "\"
  End If

  ' Write selected directory to the ini file
  iniF.StringValue( "Scripts", "LastDB_AuditDir") = path
  Set iniF = Nothing
Since at least 1270 this code no longer works. The correct selected path is stored in mediamonkey.ini but the last used folder is not selected by default by SDB.SelectFolder.
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

Re: 1274 SDB.SelectFolder

Post by Owyn »

This problem can be seen using the standard script Tools->Scripts->Export Subscribed Podcasts
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
rusty
Posts: 9003
Joined: Tue Apr 29, 2003 3:39 am
Location: Montreal, Canada

Re: 1274 SDB.SelectFolder

Post by rusty »

Ludek
Posts: 5107
Joined: Fri Mar 09, 2007 9:00 am

Re: 1274 SDB.SelectFolder [#6056]

Post by Ludek »

Owyn, Rusty, I cannot reproduce here with 1275.
'Export subscribed podcasts' script works fine for me and it remember the last used path also after restarting of MM.

Could you please re-test and tell me whether it is specific to a kind of path, e.g. network paths only?

Thx.
Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

Re: 1274 SDB.SelectFolder [#6056]

Post by Owyn »

Code: Select all

ExportOPML.vbs
--------------
  ' Let user select the output path
  Dim path
  path = iniF.StringValue( "Scripts", "LastExportOPMLDir")

  path = SDB.SelectFolder( path, SDB.Localize( "Select where to export the OPML file."))
---------------

MediaMonkey.ini
---------------
[Scripts]
LastExportOPMLDir=C:\Users\Ian\Documents\
---------------
The default report directory definitely exists.

When I run the script the selection dialog starts with no directory selected.

Edit: Added screen shot.
Image
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
Ludek
Posts: 5107
Joined: Fri Mar 09, 2007 9:00 am

Re: 1274 SDB.SelectFolder [#6056]

Post by Ludek »

I cannot reproduce, but I see one reason: I use Win XP Pro 32 and you uses Vista or Win7 64x, don't you?
Last edited by Ludek on Mon Oct 19, 2009 8:42 am, edited 2 times in total.
Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

Re: 1274 SDB.SelectFolder [#6056]

Post by Owyn »

Yep. Vista Home Premium SP2 (see signature for fulll details).
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
Ludek
Posts: 5107
Joined: Fri Mar 09, 2007 9:00 am

Re: 1274 SDB.SelectFolder [#6056]

Post by Ludek »

Edit: Could you please test by using this MediaMonkey.exe: http://rapidshare.com/files/295052285/M ... y.exe.html
Thx.
Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

Re: 1274 SDB.SelectFolder [#6056]

Post by Owyn »

No change. I double checked that About shows 1274 instead of 1275 that I had been running.
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
Ludek
Posts: 5107
Joined: Fri Mar 09, 2007 9:00 am

Re: 1274 SDB.SelectFolder [#6056]

Post by Ludek »

Ok, it seems to be related to Vista and to the user folders. Could you confirm that it is related only the the C:\User folders and that for e.g. E:\MMExports\OPML\ directory it works fine?
Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

Re: 1274 SDB.SelectFolder [#6056]

Post by Owyn »

Created "C:\MMDATA\Reports" folder.
Exported OPML to that folder (did not test new folder in dialog).
Reran Export script. Default now used properly.
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
rusty
Posts: 9003
Joined: Tue Apr 29, 2003 3:39 am
Location: Montreal, Canada

Re: 1274 SDB.SelectFolder [#6056]

Post by rusty »

Fixed in 1276.

-Rusty
Owyn
Posts: 2018
Joined: Fri Mar 21, 2008 10:55 am
Location: Canada

Re: 1274 SDB.SelectFolder [#6056]

Post by Owyn »

Verified in 1276. Thanks.
Cogito cogito ergo cogito sum. (Ambrose Bierce)
I drink therefore I am. (Monty Python)
Vista 32bit Home Premium SP2 / MM3.2.1.1297 Gold / Last.Fm 1.0.2.22 / IE8
Dell Inspiron 530 (1.8 Core2 / 2GB)
Skin: Vitreous Blue
Scripts: Add/Remove Playstat|Advanced Duplicate Find & Fix|Album Art Tagger|Backup|Batch Art Finder|Calculate Cover Size|Case&Leading Zero Fixer|DB_Audit|DB_Clean|DB_Schema|Event Logger|Genre Finder|Lyricator|Magic Nodes|MM2VLC|Monkey Rok|MusicIP Tagger|PUID Generator|RegExp Find & Replace|Right Click for Scripts|Scriptreloader|SQL Viewer|Stats(Filtered)|Tagging Inconsistencies
Post Reply