Multiple not-siteorigin widgets not saving correct

Hi Alex,

It’s a standard select with multiple indeed. The widget class is the same as the wordpress tutorial, just with the extra field.

I’ve added two screenshots of the widget, after it’s saved. I output the $instance[‘selectfield’] in both cases.
Normal widget area:

Siteorigin widget area:

Note: if I save a second instance in the builder, the first gets emptied.

Sorry, images are not working. New try on imgur

Yes, of course:

<?php
class Vdb_Ratio_Widget extends WP_Widget {

	/**
	 * The ID of this plugin.
	 *
	 * @since    1.0.0
	 * @access   private
	 * @var      string    $plugin_name    The ID of this plugin.
	 */
	private $plugin_name;

	/**
	 * Meta prefix
	 *
	 * @since   1.0.0
	 * @access  private
	 * @var     string  $prefix
	 */
	private $prefix;

	/**
	 * Register widget with WordPress.
	 */
	function __construct() {
		$this->plugin_name = 'vdb-ratio';
		$this->prefix      = $this->plugin_name.'-';

		parent::__construct(
			$this->prefix.'widget',
			esc_html__( 'Van der Biesen - Ratio Widget', $this->plugin_name ),
			array( 'description' => esc_html__( 'A nice Widget', $this->plugin_name ), )
		);
	}

	private function get_trips($ids = []){
		$args = [
			'post_type' => $this->prefix.'trip',
			'posts_per_page' => -1,
		];
		if(!empty($ids)){
			if(!is_array($ids)){
				$ids = [$ids];
			}
			$args['post__in'] = $ids;
		}
		return new WP_Query($args);
	}

	/**
	 * Front-end display of widget.
	 *
	 * @see WP_Widget::widget()
	 *
	 * @param array $args     Widget arguments.
	 * @param array $instance Saved values from database.
	 */
	public function widget( $args, $instance ) {
	    //echo "<pre>".print_r($instance['trips'],1)."</pre>";
		echo $args['before_widget'];
		if ( ! empty( $instance['title'] ) ) {
			echo $args['before_title'] . apply_filters( 'widget_title', $instance['title'] ) . $args['after_title'];
		}
		if ( ! empty( $instance['trips'] ) ) {
			$trips = $this->get_trips($instance['trips']);?>
			<div class="<?php echo $this->prefix?>trips">
				<?php foreach($trips->posts as $trip){ ?>
					<div class="<?php echo $this->prefix?>trips__trip">
						<div class="<?php echo $this->prefix?>trips__trip__image">
							<?php echo wp_get_attachment_image(get_post_thumbnail_id($trip->ID), 'medium'); ?>
						</div>
						<div class="<?php echo $this->prefix?>trips__trip__content">
							<div class="<?php echo $this->prefix?>trips__trip__content__title">
								<?php echo $trip->post_title;?>
							</div>
							<div class="<?php echo $this->prefix?>trips__trip__content__depart-from">
								<?php echo __('Leaving: ',$this->plugin_name) . date('d-m-Y',strtotime(get_post_meta($trip->ID,$this->prefix.'depart-from',true))); ?>
							</div>
							<div class="<?php echo $this->prefix?>trips__trip__content__price-from">
								€ <?php echo get_post_meta($trip->ID,$this->prefix.'price-from', true); ?>
							</div>
							<a href="<?php the_permalink($trip->ID) ?>"><?php _e('View',$this->plugin_name) ?></a>
						</div>
					</div>
					<?php
				}?>
			</div>
			<?php
		}

		echo $args['after_widget'];
	}

	/**
	 * Back-end widget form.
	 *
	 * @see WP_Widget::form()
	 *
	 * @param array $instance Previously saved values from database.
	 */
	public function form( $instance ) {
		//echo "<pre>".print_r($instance['trips'],1)."</pre>";
		$title = ! empty( $instance['title'] ) ? $instance['title'] : esc_html__( 'New title', $this->plugin_name );
		$trips = ! empty( $instance['trips'] ) ? $instance['trips'] : [];
		$trips_list = $this->get_trips();
		?>
		
			<label for="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>"><?php esc_attr_e( 'Title:', $this->plugin_name ); ?></label>
			<input class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'title' ) ); ?>" type="text" value="<?php echo esc_attr( $title ); ?>">
		


		
			<label for="<?php echo esc_attr( $this->get_field_id( 'trips' ) ); ?>"><?php esc_attr_e( 'Trips:', $this->plugin_name ); ?></label>
			<select size="10" multiple="multiple" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'trips' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'trips' ) ); ?>[]">
				<?php foreach($trips_list->posts as $item): ?>
					<?php $selected = (in_array($item->ID,$trips))? 'selected="selected"':''; ?>
					<option <?php echo $selected ?> value="<?php echo $item->ID ?>"><?php echo $item->post_title ?></option>
				<?php endforeach;?>
			</select>
		


		<?php
	}

	/**
	 * Sanitize widget form values as they are saved.
	 *
	 * @see WP_Widget::update()
	 *
	 * @param array $new_instance Values just sent to be saved.
	 * @param array $old_instance Previously saved values from database.
	 *
	 * @return array Updated safe values to be saved.
	 */
	public function update( $new_instance, $old_instance ) {
		$instance = array();
		$instance['title'] = ( ! empty( $new_instance['title'] ) ) ? sanitize_text_field( $new_instance['title'] ) : '';
		$instance['trips'] = ( ! empty( $new_instance['trips'] ) ) ? esc_sql( $new_instance['trips'] ) : '';
		return $instance;
	}

}

Hmm the builder is now processing it correct, but the original widget isn’t finding an array as posted value, so only saves one single value.
I’ll dig into that later, at least the builder is working now and I know where’s the trouble :)

Thanks!

I know, I’ve made a lot of SiteOrigin widgets for a client (I love how easy that is by the way :) ). But this is a separate plugin not specific for the SiteOrigin builder.