Skip to content
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

[Bug 19121] Pass correct folder to standaloneSaved msg#5106

Merged
livecodepanos merged 2 commits into
livecode:release-8.1.3from
livecodepanos:bugfix-19121
Jan 24, 2017
Merged

[Bug 19121] Pass correct folder to standaloneSaved msg#5106
livecodepanos merged 2 commits into
livecode:release-8.1.3from
livecodepanos:bugfix-19121

Conversation

@livecodepanos

Copy link
Copy Markdown
Contributor

If the user chooses to save a standalone named "MyApp" in Desktop folder, then a folder is passed to standaloneSaved message.

In LC 8.1.3 RC-1 this folder was path/to/Desktop/MyApp/MacOSX (not correct), and in pre-LC 8.1.3 RC-1 this was path/to/Desktop/ (not very useful).

In this patch the folder will now be path/to/Desktop/MyApp/.

This ensures that when having multiple copies of the same standalone, in folders named "MyApp", "MyApp1", "MyApp2" etc, the path to the most recent one will be passed to standaloneSaved message, e.g. path/to/Desktop/MyApp5/

@livecodepanos livecodepanos added this to the 8.1.3-rc-2 milestone Jan 24, 2017
@peter-b

peter-b commented Jan 24, 2017

Copy link
Copy Markdown
Contributor

@livecode-vulcan review ok cbd8874

@livecode-vulcan

Copy link
Copy Markdown
Contributor

💙 review by @peter-b ok cbd8874

livecode-vulcan added a commit that referenced this pull request Jan 24, 2017
[Bug 19121] Pass correct folder to standaloneSaved msg

If the user chooses to save a standalone named "MyApp" in Desktop folder, then a folder is passed to `standaloneSaved` message.

In LC 8.1.3 RC-1 this folder was `path/to/Desktop/MyApp/MacOSX` (not correct), and in pre-LC 8.1.3 RC-1 this was `path/to/Desktop/` (not very useful).

In this patch the folder will now be `path/to/Desktop/MyApp/`.

This ensures that when having multiple copies of the same standalone, in folders named "MyApp", "MyApp1", "MyApp2" etc, the path to the most recent one will be passed to `standaloneSaved` message, e.g. `path/to/Desktop/MyApp5/`
@livecode-vulcan

Copy link
Copy Markdown
Contributor

😎 test success cbd8874

@livecodepanos livecodepanos merged commit f9aba90 into livecode:release-8.1.3 Jan 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants