Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

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

Jump to bottom

Zrušení probíhajícího xhr requestu při zavření pdboxu #7

Closed
2 tasks done
zipper opened this issue Oct 19, 2017 · 4 comments
Closed
2 tasks done

Zrušení probíhajícího xhr requestu při zavření pdboxu #7

zipper opened this issue Oct 19, 2017 · 4 comments
Assignees
@zipper

Comments

@zipper
Copy link
Contributor

zipper commented Oct 19, 2017 •
edited

Při zavření pdboxu dochází k zrušení probíhajícího requestu. Nynější implementace má dva problémy:

  • Při zavření pdboxu se zruší i xhr request, který s pdboxem nemusí souviset (probíhající na pozadí na základě akce v pdboxu, např. přepočítávání košíku pod pdboxem).
  • Při každém requestu se naváže nový callback po zrušení xhr, i když je daný xhr dávno dokončený. Je potřeba toto upravit podobným způsobem jako pushOriginalApplied nebo jako rušení xhr v extension abort.
@zipper zipper self-assigned this Oct 19, 2017
@zipper
Copy link
Contributor Author

zipper commented Oct 30, 2017

Po úpravách teoreticky řešitelné přes vypnutí extension pdbox na daném AJAXu. Je nutné upravit chování automatického přidávání class na vše (filtr na vypnuté extension) a navázání abort na afterClose.

@zipper zipper closed this as completed in 9b224a4 Nov 15, 2017
zipper added a commit that referenced this issue Nov 15, 2017
@zipper
- closes #7
9ef8bbe
@Spilky Spilky reopened this Jan 31, 2018
@Spilky
Copy link

Spilky commented Jan 31, 2018

V tuto chvíli není možné vypnout extension unique, protože request je zrušen vždy v extension pdbox.

zipper added a commit that referenced this issue Mar 7, 2019
@zipper
#7 Zrušení probíhajícího xhr requestu při zavření pdboxu
9d04628 
- Do extension se neukládá každý `xhr`, ale pouze ty, které opravdu souvisí s `pdboxem`. Stejně tak k rušení dojde pouze v případě, že oba requesty (probíhající i nový) souvisí s `pdboxem`.
@zipper zipper mentioned this issue Mar 8, 2019
Merged
@zipper
Copy link
Contributor Author

zipper commented Mar 8, 2019

Mělo by být opraveno v PR #14.

@zipper
Copy link
Contributor Author

zipper commented Mar 29, 2019

Vydáno ve verzi 1.4.0, viz https://github.com/peckadesign/pd.ajax/releases/tag/1.4.0.

@zipper zipper closed this as completed Mar 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zipper zipper

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zipper @Spilky