Get term_id for each instance of custom taxonomy

Question

I am building a custom image slider.

I have a custom post type called 'slide' with an associated custom taxonomy called 'slideshows'.

When you create a new 'slideshow' it add to a custom table in the DB called wp_slideshowsmeta with columns for "meta_id, slideshows_id, meta_key, and meta_value".

Each slideshow has its own settings page using 'slideshows_add_form_fields' and 'slideshows_edit_form_fields'.

To call a slideshow I created a function ( 'gps_slider()' ) that is placed in the template file where you want the slideshow to appear. It has one parameter which is the slideshow name.

Everything works great... it loads the appropriate slideshow and functions. BUT, all the slideshows use the first one's settings.

I need to pass the slideshow name to my function that uses 'wp_localize_script' to pass the settings to the javascript.

I'm guessing it's likely a foreach loop, but can't seem to see how to construct it.

Any help is greatly appreciated.

Bryce

Answer 1

The structure of this is kind of weird. But your main issue is that the wp_enqueue_scripts action runs before any calls to gps_slider() can possibly happen. Hook your script localization to the wp_footer action instead, so the slideshow data in the page already exists and has been output. Additionally you'll have to make your slideshow data an array of arrays so you can pass the settings for each individually.

$data = array();
$data[$name] = array(
    'gpsslidermode' => $slider_mode,
    'gpssliderspeed' => $slider_speed
);
wp_localize_script('gps-slider_script', 'gpsslidersettings', $data);

Also, why not just use the options table for your slideshow data, keyed to the slideshow ID? You don't seem to be doing anything with your custom table that couldn't be replicated with a built in core table, except creating more work for yourself. At the very least just save all of your slideshow meta in a single row in a serialized array, and cut 8 database queries from each slideshow load.

EDIT-

function gps_slider( $slideshow = '' ) {
    include('lib/gps-slider-output.php');
    global $slideshows;
    $slideshows[$slideshow] = array(
        // fetch and insert your settings data here
        'gpsslidermode' => $slider_mode,
        'gpssliderspeed' => $slider_speed,
        // etc..
    );
}

function gps_slider_localize(){
    global $slideshows;
    wp_localize_script('gps-slider_script', 'gpsslidersettings', $slideshows);
}
add_action( 'wp_footer', 'gps_slider_localize' );

Answer 2

I fear it may be because on line 8 of your pastebin, inside the get_term_by function you have hard coded 'slider95' instead of using the variable $name that you pass over into your function.

All its ever going to do is get the details for just slider95.