• Resolved lircabin

    (@lircabin)


    Hello there,

    I have no errors happening, not on a debug log for the php neither for the javascript but only the last selected value on the dynamic dropdown with properties multiple and select2 clas gets parsed to the email.

    Is there any further configuration I should do for such fields?
    For the moment I am going to use a hidden field and stringify the value of the field (from array to comma separated value) but I think this should be further investigated.

    With thanks,
    Matheus Alves

Viewing 12 replies - 1 through 12 (of 12 total)
  • Plugin Author Aurovrata Venet

    (@aurovrata)

    but only the last selected value on the dynamic dropdown with properties multiple and select2 clas gets parsed to the email.

    hahaha, yes, that’s a bug alright!

    However, in the latest version of the plugin, you can now filter individual mail tags (see screenshot #23), and the filter has access to all submitted values, so you can reconstruct the mailtag shortcut to include your missing values.

    In the next release I will fix this.

    Thread Starter lircabin

    (@lircabin)

    However, in the latest version of the plugin, you can now filter individual mail tags (see screenshot #23)

    Oh! I didn’t realize this would work. It’s a good fix for the moment.

    Thanks for this awesome plugin btw!

    • This reply was modified 4 years, 9 months ago by lircabin.
    Plugin Author Aurovrata Venet

    (@aurovrata)

    @aurovrata I think it starts with the html select input not having the array symbol in the name attribute, therefore the posted data gets replaced over and over until the last one. From there might be able to parse it same as a checkbox is parsed.

    Indeed, I realise that’s an important missing bit. I will fix and and make a minor release before end of day.

    Plugin Author Aurovrata Venet

    (@aurovrata)

    released v3.3.1 with the multiple select fixed.

    Plugin Author Aurovrata Venet

    (@aurovrata)

    Thanks for this awesome plugin btw!

    glad you like it. Do leave a review when you a moment to spare.

    Plugin Author Aurovrata Venet

    (@aurovrata)

    so apparenlty the repo has an issue and the plugin update is not showing up on the front-end. Do you see it reflected in your dashboard?

    WP plugin isn’t sure about how it will take…

    btw, I saw you’re in Brazil, I hope you and your dev team are ok given the pandemic spread in your country.

    Stay safe!

    Thread Starter lircabin

    (@lircabin)

    released v3.3.1 with the multiple select fixed.

    Wow that was quick! Thank you very much

    so apparenlty the repo has an issue and the plugin update is not showing up on the front-end. Do you see it reflected in your dashboard?

    Yes! It didn’t show it yet! Will update it as soon as it does and give it a go!

    Thanks for the concern! Yes, things are really bad right now over here but coding all day has helped.

    Hope you are well over there too!

    Plugin Author Aurovrata Venet

    (@aurovrata)

    Hope you are well over there too!

    I am currently based in Delhi, India and things are also getting worse….thank god for computers and coding, time flies even in confinement ??

    Plugin Author Aurovrata Venet

    (@aurovrata)

    the trac repo is reflecting the file changes, you need only this one to get the update, in case you’re in a hurry.

    Thread Starter lircabin

    (@lircabin)

    the trac repo is reflecting the file changes, you need only this one to get the update, in case you’re in a hurry.

    @aurovrata
    Awesome! I did update it and I tried it out. It does get all of the fields now.
    I haven’t had the time to go through the whole logic and code of the plugin as of yet. But if the dynamic_select is required (dynamic_select*) the validation used in the plugin doesn’t work and returns the message “The field is required” even if the user has selected options. I traced it back to about line 1147 in the “public/class-cf7-grid-layout-public”.
    There’s probably a more elegant solution for this but for me, at the moment: adding a check that if the filed type is dynamic_select* to validate as checkbox* seemed to work for me as the multiple select has essentially the same data structure as a checkbox.

    Plugin Author Aurovrata Venet

    (@aurovrata)

    oh, that’s interesting! I will look fix this too. Thanks for the debugging info!

    PS: The plugin is maintained in this GitHub repo, which is easier to report bugs and annotate lines of code, in case you find more bugs ??

    Thread Starter lircabin

    (@lircabin)

    oh, that’s interesting! I will look fix this too. Thanks for the debugging info!

    You are welcome!

    Thanks for the GitHub Repo! gonna add a watch over there!
    Gonna mark this as resolved, thanks for all the info and support @aurovrata !

Viewing 12 replies - 1 through 12 (of 12 total)
  • The topic ‘Dynamic Dropdown Multiple Select Issue’ is closed to new replies.