Merge pull request #207 from BoldGrid/pro-issue-137

fixes pro issue #137

Author: avonville

assets/spa/components/form-builder/index.js

@@ -232,6 +232,7 @@ weForms.routeComponents.FormEditComponent = {
                 },
 
                 success: function (response) {
+                    self.$store.state.settings = response.settings;
                     if (response.form_fields) {
                         self.$store.commit('set_form_fields', response.form_fields);
                     }

includes/class-ajax.php

@@ -109,9 +109,10 @@ public function save_form() {
         $integrations  = array();
 
         if ( isset( $post_data['settings'] ) ) {
-            $settings = (array) json_decode( $post_data['settings'] );
-            $settings['message'] = sanitize_text_field( $settings['message'] );
-            error_log(print_r($settings, true));
+            $settings                  = json_decode( $post_data['settings'], true );
+            $settings['message']       = sanitize_text_field( $settings['message'] );
+            $settings['url']           = sanitize_url( $settings['url'] );
+            $settings['limit_message'] = sanitize_text_field( $settings['limit_message'] );
         } else {
             $settings = isset( $form_data['wpuf_settings'] ) ? $form_data['wpuf_settings'] : [];
         }
@@ -142,7 +143,12 @@ public function save_form() {
 
         do_action( 'weforms_update_form', $form_data['wpuf_form_id'], $form_fields, $settings );
 
-        wp_send_json_success( [ 'form_fields' => $form_fields ] );
+        wp_send_json_success(
+            array(
+                'form_fields' => $form_fields,
+                'settings'    => $settings,
+            )
+        );
     }
 
     /**