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.

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

Fail to seek data from yiiActiveForm('data') #300

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

agungsijawir
Copy link

Sometimes script failed to seek settings data from yiiActiveForm('data'), thus will render errors on forms.

This patch fixes the issue by checking if typeof yiiActiveFormData is not undefined, allow seeking from data, otherwise use default Yii error/success, CSS Class.

Fix if the parsing method failed to seek settings data from `yiiActiveForm('data')`.
Will use default CSS classes and seek settings only if the object is not undefined.
minifying edited JS
@agungsijawir
Copy link
Author

Ah btw, I am using Yii version 2.0.28 as per I create this PR.

@agungsijawir
Copy link
Author

Maybe @wbraganca can take a look for this.

@agungsijawir
Copy link
Author

Related to #258 #293

@AnniG
Copy link

AnniG commented Oct 25, 2020

Hello @wbraganca and @agungsijawir,
thank you for this helpful extension! Is there a reason why this pull request is not merged yet? For me it works perfectly.

@agungsijawir
Copy link
Author

agungsijawir commented Oct 26, 2020

Hello @AnniG

Thanks for reaching me out. I am curious that my PR still not merged nor get any feedback. It looks like the author is away.

If somehow my patch works for you, thank you.

StitchUA added a commit to StitchUA/yii2-dynamicform that referenced this pull request Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants