Patches item #3011815, was opened at 2010-06-05 11:15 Message generated for change (Comment added) made by eighthave You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3011815...

Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: pd-extended Group: None

Status: Pending

Resolution: Accepted Priority: 5 Private: No Submitted By: Dan Wilcox (danomatika) Assigned to: Hans-Christoph Steiner (eighthave) Summary: Path editor cannot edit bad paths

Initial Comment: Running Pd-Extended 0.42.5 rc1 on Mac OSX 10.6.3

I moved some local lib folders and tried to update them in the Path editor. Selecting one of these paths and hitting Edit... results in a TK error dialog with the following message: Error: Bad directory "/Users/path/to/old/folder...". I cannot edit the path within PD, which is annoying.

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave)

Date: 2010-06-06 19:28

Message: Oops, sorry about the typo...

I think parsing to the nearest good location could be nice, but too many error messages gets confusing. I don't think this particular error message would change the user's behavior, so it seems to fall on the "too much" side.

----------------------------------------------------------------------

Comment By: Dan Wilcox (danomatika) Date: 2010-06-06 18:44

Message: Looks good, although you spelled my name wrong :D

Also, I added the console print for feedback as to why you'd be looking at a dialog box in the home dir instead of the given path. Ideally, the bad path should be parsed to find the nearest good location. For instance, if the end folder name has been changed, the dialog box *should* start one level down.

The current version is a quick fix so at least the editor is useable in this case.

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave) Date: 2010-06-06 18:29

Message: Accepted in an edited form, the error message seemed too much for such a little thing. I also added it to 0.43 while I was at it. Here's the commit:

http://pure-data.svn.sourceforge.net/viewvc/pure-data?view=rev&revision=...

----------------------------------------------------------------------

Comment By: Dan Wilcox (danomatika) Date: 2010-06-05 12:06

Message: I added PathEditor.patch for pd-tk which adds a check on existing paths before editing them. If they don't exist, an error message is printed to ::pdwindow::error and the dialog path is set to the home dir.

----------------------------------------------------------------------

You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3011815...