Bugfix: Dispose Issue

[schillingdavid]

Fixed an issue with the disposing of the animation controller which could cause memory leaks and crashes.

[schillingdavid]

Fixes the following exception:

The following assertion was thrown while finalizing the widget tree:
_FlutterPasswordStrengthState#61795(ticker active) was disposed with an active Ticker.

_FlutterPasswordStrengthState created a Ticker via its SingleTickerProviderStateMixin, but at the time dispose() was called on the mixin, that Ticker was still active. The Ticker must be disposed before calling super.dispose().

Tickers used by AnimationControllers should be disposed by calling dispose() on the AnimationController itself. Otherwise, the ticker will leak.

[volmus]

Any news on this @JinHoSo ?

[JinHoSo]

@schillingdavid Thank you 🙏

[schillingdavid]

@schillingdavid Thank you 🙏

Of course! 😊 Do you plan on tagging a new release real soon? 😊 If so, what do you think when? 😬

[arielt]

Hi , looks like the fix was reverted in one of the later commits: d28089d
Any chance to restore it?
The issue with the Ticker is seen again.

[schillingdavid]

Hi , looks like the fix was reverted in one of the later commits: d28089d Any chance to restore it? The issue with the Ticker is seen again.

Yeah, it really was reverted. I added a new Pull Request #17.
It'll be merged after @JinHoSo reviewed and accepted it. 😊

[arielt]

Checking back, I think it is still a useful fix.

[robyf]

Any news on this?