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

[[ Bug 22689 ]] Fix execution error when calling JavaScript in iOS browser widget #7328

Merged
livecodepanos merged 3 commits into
livecode:developfrom
livecodeian:bugfix-22689
Apr 21, 2020
Merged

[[ Bug 22689 ]] Fix execution error when calling JavaScript in iOS browser widget #7328
livecodepanos merged 3 commits into
livecode:developfrom
livecodeian:bugfix-22689

Conversation

@livecodeian

Copy link
Copy Markdown
Contributor

This patch fixes bug 22689, where calling JavaScript in a iOS
browser widget would result in an execution error if the JavaScript does not
return a value.

Closes https://quality.livecode.com/show_bug.cgi?id=22689

Ian Macphail added 3 commits April 20, 2020 14:42
This patch adds the "optional" keyword to foreign handler declarations in the
browser library where the out param type is a pointer that may be null.
… value

This patch fixes a bug in libbrowser where null would be returned when
evaluating a JavaScript statement with no result value. The empty string will
be returned instead.
This patch adds a release note for bug 22689, where calling JavaScript in a iOS
browser widget would result in an execution error if the JavaScript does not
return a value.
@livecodeian livecodeian added this to the 9.6.0-rc-1 milestone Apr 20, 2020
@livecodeian livecodeian requested a review from runrevmark April 20, 2020 13:55
@livecodeian livecodeian self-assigned this Apr 20, 2020
@runrevmark

Copy link
Copy Markdown
Contributor

@livecode-vulcan review ok e612558

@livecode-vulcan

Copy link
Copy Markdown
Contributor

💙 review by @runrevmark ok e612558

livecode-vulcan added a commit that referenced this pull request Apr 21, 2020
[[ Bug 22689 ]] Fix execution error when calling JavaScript in iOS browser widget

This patch fixes bug 22689, where calling JavaScript in a iOS
browser widget would result in an execution error if the JavaScript does not
return a value.

Closes https://quality.livecode.com/show_bug.cgi?id=22689
@livecode-vulcan

Copy link
Copy Markdown
Contributor

😎 test success e612558

  • try-community-armv6-android-sdk26_ndk16r15: success
  • try-community-armv7-android-ndk16r15: success
  • try-community-arm64-android-ndk16r15: success
  • try-community-x86-android-ndk16r15: success
  • try-community-x86_64-android-ndk16r15: success
  • try-community-js-emscripten-sdk1.35: success
  • try-community-universal-ios-iphoneos13.4: success
  • try-community-universal-ios-iphonesimulator13.4: success
  • try-community-universal-mac-macosx10.9: success
  • try-community-x86-linux-debian8: success
  • try-community-x86_64-linux-debian8: success
  • try-community-x86-win32: success
  • try-community-x86_64-win32: success

@livecodepanos livecodepanos merged commit 1a44088 into livecode:develop Apr 21, 2020
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.

4 participants