Error with Fluent Forms

  • Resolved undfine

    (@undfine)


    3 months, 2 weeks ago

    Hello and thanks for creating this plugin!

    It’s like almost exactly what I’m looking for: first and last-touch attribution with server-side tracking and the consent feature is a bonus.

    However, there are a few issues:

    Settings page — For some reason the checkbox settings aren’t saving to the DB. I can only set the “enable_attribution” and “cookie_days”.

    Fluentforms integration caused a fatal error — Action should only accept 1 argument:
    https://fluentforms.com/docs/fluentform_form_element_start/

    FILE: class-fluent-forms-adapter.php (line 45) 

    add_action( 'fluentform_form_element_start', array( $this, 'add_hidden_fields' ), 10, 2 );

    Thanks!

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author Hugo Carvalho

    (@hugoc)

    2 months, 3 weeks ago

    Thanks for the detailed report and for linking the Fluent Forms hook docs.

    I was able to confirm both issues.

    1. The settings page problem was on my side. The checkbox-based attribution settings were not being saved reliably, which is why only enable_attribution and cookie_days were persisting. I’ve corrected the settings registration/sanitization so those checkbox values save properly.
    2. You are also correct about the Fluent Forms fatal error. The adapter was registering fluentform_form_element_start with accepted_args = 2, but that action only passes 1 argument. I’ve updated the hook registration and callback signature accordingly.

    These fixes will be included in the next plugin update.

    Thanks again for catching both issues.

    Thread Starter undfine

    (@undfine)

    2 months, 2 weeks ago

    Thanks for the reply and fixing those issues!

Viewing 2 replies - 1 through 2 (of 2 total)

You must be logged in to reply to this topic.