We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
indicatorのオプションでpromsieを渡すとそのpromiseのついてのcommonFailHandler(以下CFH)が動作しなくなる。
// CFH設定済み var dfd = h5.async.deferred(); var p = dfd.promise(); var indicator = h5.ui.indicator(document, {promises: p}); dfd.reject(); // CFHが動作しない
FW内部でfailハンドラが付加され、CFHを呼び出すフラグが無くなってしまうためである。 #210 で追加した_h5UnwrappedCallを使用するよう修正する。
_h5UnwrappedCall
The text was updated successfully, but these errors were encountered:
ver.1.1.4では、Deferredの実装のバグによりindicatorにpromiseを渡しても、そのpromiseにfailハンドラを設定しなければCFHが動作していた。 ver.1.1.5でDeferredの実装が変わったため、promiseを渡すと常にCFHが動作しないようになった。
indicatorに画面表示を自動的に消すように設定しただけでCFHが動作しなくなる挙動は 開発者にとって使いづらい場面の方が多いので、indicatorの仕様として CFHの動作を阻害しないことを明確にし、そのように動作するようにする。 (結果として、ver.1.1.4までの動作に戻ることとなる。)
Sorry, something went wrong.
hifive#211 対応しました。テストケース追加しました。
a698387
fukudayasuo
No branches or pull requests
indicatorのオプションでpromsieを渡すとそのpromiseのついてのcommonFailHandler(以下CFH)が動作しなくなる。
FW内部でfailハンドラが付加され、CFHを呼び出すフラグが無くなってしまうためである。
#210 で追加した
_h5UnwrappedCall
を使用するよう修正する。The text was updated successfully, but these errors were encountered: