I'm trying to extend a function of a WordPress plugin to accept another parameter but I'm having problems figuring out the output. The plugin I'm talking about is NextGen Gallery and I'm trying to make it to show a slideshow of images with descriptions.
The code I'm using to show the slideshow on my theme file is this:
<?php echo nggShow_JS_Slideshow(1,906,358,'caption'); ?>
Below is the complete function:
function nggShow_JS_Slideshow($galleryID, $width, $height, $template = '', $class = 'ngg-slideshow') {
global $slideCounter;
$ngg_options = nggGallery::get_option('ngg_options');
// we need to know the current page id
$current_page = (get_the_ID() == false) ? rand(5, 15) : get_the_ID();
// look for a other slideshow instance
if ( !isset($slideCounter) ) $slideCounter = 1;
// create unique anchor
$anchor = 'ngg-slideshow-' . $galleryID . '-' . $current_page . '-' . $slideCounter++;
if (empty($width) ) $width = (int) $ngg_options['irWidth'];
if (empty($height)) $height = (int) $ngg_options['irHeight'];
//filter to resize images for mobile browser
list($width, $height) = apply_filters('ngg_slideshow_size', array( $width, $height ) );
$width = (int) $width;
$height = (int) $height;
$out = '<div id="' . $anchor . '" class="' . $class . '" style="height:' . $height . 'px;width:' . $width . 'px;">';
$out .= "\n". '<div id="' . $anchor . '-loader" class="ngg-slideshow-loader" style="height:' . $height . 'px;width:' . $width . 'px;">';
$out .= "\n". '<img src="'. NGGALLERY_URLPATH . 'images/loader.gif" alt="" />';
$out .= "\n". '</div>';
$out .= '</div>'."\n";
$out .= "\n".'<script type="text/javascript" defer="defer">';
$out .= "\n" . 'jQuery(document).ready(function(){ ' . "\n" . 'jQuery("#' . $anchor . '").nggSlideshow( {' .
'id: ' . $galleryID . ',' .
'fx:"' . $ngg_options['slideFx'] . '",' .
'width:' . $width . ',' .
'height:' . $height . ',' .
'template:' . $template . ',' .
'domain: "' . trailingslashit ( home_url() ) . '",' .
'timeout:' . $ngg_options['irRotatetime'] * 1000 .
'});' . "\n" . '});';
$out .= "\n".'</script>';
return $out;
}
The problem is the 'template' parameter that I've added to the function. It doesn't work for some reason - the slideshow keeps loading forever. You can see the full nggfunctions.php file here. And the website were you can see this live is this. I also should mention that PHP is not one of my strong points.
Later edit: isn't there a way to add the description of each image directly in the function and not pass it through the 'template' parameter?
Related
Trying to display filename but code is resulting in full file url displaying
if (trim($s) == "") continue;
$arrfilename = explode("\/", $s);
$shortfilename = $arrfilename[count($arrfilename)-1];
$path_parts = pathinfo($s);
$dir = $path_parts['dirname'];
$basename = $path_parts['basename'];
$ext = $path_parts['extension'];
$fn = $path_parts['filename'];
$sliderimage = $dir . '/' . $fn . '.' . $ext;
if (!file_exists($sliderimage) && !file_exists('../' . $sliderimage)) $sliderimage = $s;
$output .= '[setslideshowlinkattributes ssrs="' . $s . '"]<img src="/' . $sliderimage . '" alt="' .$shortfilename . '" /></a>';
}
$output .= '</div>';
$output .= ' <div id="htmlcaption" style="display: inline;">' . $this->options->slideshowcaption . '</div>';
$output .= '</div>';
I'm trying to add style, depending on whether the checkbox is clicked. This code is missing "id" and "for" for input and label (line 11). Logical decision to add generation of numbers. How to do it correctly?
foreach ($choices as $choice) {
$attr = '';
$key_val = explode("|", $choice);
/* It has to be two items ( Value => Label ), otherwise don't proceed */
if (count($key_val) == 2) {
if (in_array(trim($key_val[0]), $defaults)) {
$attr = 'checked';
}
/* For admin field, we don't need <li></li> wrapper */
$html .= (($_ptype != "wccaf") ? '<li>' : '') . '<input type="checkbox" data-has_field_rules="'.$has_field_rules.'" data-is_pricing_rules="'.$_is_pricing_rules.'" class="' . $_ptype . '-field ' . $_class . '" name="' . esc_attr($_meta["name"] . $_index) . '[]" value="' . esc_attr(trim($key_val[0])) . '" ' . $attr . ' ' . $_ptype . '-type="checkbox" ' . $_ptype . '-pattern="mandatory" ' . $_ptype . '-mandatory="' . $_meta["required"] . '" ' . $_readonly . ' /><label class="wcff-option-wrapper-label">' . esc_attr(trim($key_val[1])) . '</label>' . (($_ptype != "wccaf") ? '</li>' : '');
}
}
Updated
Try the following (untested), that will add a for attribute + value to the <label> and an id attribute + value to the <input>:
foreach ($choices as $choice) {
$attr = '';
$key_val = explode("|", $choice);
/* It has to be two items ( Value => Label ), otherwise don't proceed */
if (count($key_val) == 2) {
if (in_array(trim($key_val[0]), $defaults)) {
$attr = 'checked';
}
$sprintf = sprintf( '<input type="checkbox" %s %s %s %s %s %s %s /><label %s class="wcff-option-wrapper-label">%s</label>',
'id="' . esc_attr($_meta["name"] . $_index) . '"',
'data-has_field_rules="'.$has_field_rules.'"',
'data-is_pricing_rules="'.$_is_pricing_rules.'"',
'class="' . $_ptype . '-field ' . $_class . '"',
'name="' . esc_attr($_meta["name"] . $_index) . '[]"',
'value="' . esc_attr(trim($key_val[0])) . '"',
$attr . ' ' . $_ptype . '-type="checkbox" ' . $_ptype . '-pattern="mandatory' . $_meta["required"] . '" ' . $_readonly,
'for="' . esc_attr($_meta["name"] . $_index) . '"',
esc_attr(trim($key_val[1]) ) );
$html .= $_ptype != "wccaf" ? '<li>'.$sprintf.'</li>' : $sprintf;
}
}
I have embedded the code in a sprintf() function to make it more readable, functional and easy to tune.
On my blog homepage, Author and date meta (Author name and date) are in link form. I want to show them only in text. When I tried to remove the link the text is also removed. Kindly help me to remove the link. Codes of display.php are
if (!function_exists('swift_meta_generator')):
/**
* Generate post meta.
*
* Prints the post meta information based on the options set in theme options page.
* Called around post titles on home page and single pages.
*
* #param array $meta post meta order set in options page.
* #param string $classes html classes for the post meta wrapper.
*
*/
function swift_meta_generator($meta, $classes)
{
$data = '<div class="entry-meta ' . $classes . '">';
$size = count($meta);
for ($i = 0; $i < $size; $i++) {
switch ($meta[$i]) {
case 'text' :
if ((current_time('timestamp', 1) - get_the_date('U')) < 86400)
$meta[$i + 1] = preg_replace('(on)', '', $meta[$i + 1]);
$data .= $meta[$i + 1];
$i++;
break;
case 'author' :
$data .= '<span class="vcard author fa-user"><a class="" href="' . esc_url(get_author_posts_url(get_the_author_meta('ID'))) . '" rel="author"><span class="fn">' . esc_attr(get_the_author()) . '</span></a></span> ';
break;
case 'author_avatar' :
$data .= get_avatar( get_the_author_meta('ID'), 16 );
$data .= ' <span class="vcard author"><a class="" href="' . esc_url(get_author_posts_url(get_the_author_meta('ID'))) . '" rel="author"><span class="fn">' . esc_attr(get_the_author()) . '</span></a></span> ';
break;
case 'date' :
if ((current_time('timestamp', 1) - get_the_date('U')) < 86400)
$date = human_time_diff(get_the_time('U'), current_time('timestamp')) . ' '.__('ago','swift');
else
$date = get_the_date();
$data .= '<span class="date updated fa-clock-o"><a class="" href="' . esc_url(get_permalink()) . '" title="' . esc_attr(get_the_time()) . '" rel="bookmark">';
$data .= '<time class="entry-date" datetime="' . esc_attr(get_the_date('c')) . '">' . esc_html($date) . '</time></a></span> ';
break;
case 'updated_on' :
if ((current_time('timestamp', 1) - get_the_modified_date('U')) < 86400)
$date = human_time_diff(get_post_modified_time('U'), current_time('timestamp')) . ' '.__('ago','swift');
else
$date = get_the_modified_date();
$data .= '<span class="date updated fa-clock-o"><a href="' . esc_url(get_permalink()) . '" title="' . esc_attr(get_post_modified_time()) . '" rel="bookmark">';
$data .= '<time class="entry-date" datetime="' . esc_attr(get_the_modified_date('c')) . '">' . esc_html($date) . '</time></a></span> ';
Replace this code with your code
if (!function_exists('swift_meta_generator')):
/**
* Generate post meta.
*
* Prints the post meta information based on the options set in theme options page.
* Called around post titles on home page and single pages.
*
* #param array $meta post meta order set in options page.
* #param string $classes html classes for the post meta wrapper.
*
*/
function swift_meta_generator($meta, $classes)
{
$data = '<div class="entry-meta ' . $classes . '">';
$size = count($meta);
for ($i = 0; $i < $size; $i++) {
switch ($meta[$i]) {
case 'text' :
if ((current_time('timestamp', 1) - get_the_date('U')) < 86400)
$meta[$i + 1] = preg_replace('(on)', '', $meta[$i + 1]);
$data .= $meta[$i + 1];
$i++;
break;
case 'author' :
$data .= '<span class="vcard author fa-user"><span class="fn">' . esc_attr(get_the_author()) . '</span></span> ';
break;
case 'author_avatar' :
$data .= get_avatar( get_the_author_meta('ID'), 16 );
$data .= ' <span class="vcard author"><a class="" href="' . esc_url(get_author_posts_url(get_the_author_meta('ID'))) . '" rel="author"><span class="fn">' . esc_attr(get_the_author()) . '</span></a></span> ';
break;
case 'date' :
if ((current_time('timestamp', 1) - get_the_date('U')) < 86400)
$date = human_time_diff(get_the_time('U'), current_time('timestamp')) . ' '.__('ago','swift');
else
$date = get_the_date();
$data .= '<span class="date updated fa-clock-o">' . esc_attr(get_the_time()) . '</span>';
$data .= '<time class="entry-date" datetime="' . esc_attr(get_the_date('c')) . '">' . esc_html($date) . '</time></span> ';
break;
case 'updated_on' :
if ((current_time('timestamp', 1) - get_the_modified_date('U')) < 86400)
$date = human_time_diff(get_post_modified_time('U'), current_time('timestamp')) . ' '.__('ago','swift');
else
$date = get_the_modified_date();
$data .= '<span class="date updated fa-clock-o"><a href="' . esc_url(get_permalink()) . '" title="' . esc_attr(get_post_modified_time()) . '" rel="bookmark">';
$data .= '<time class="entry-date" datetime="' . esc_attr(get_the_modified_date('c')) . '">' . esc_html($date) . '</time></a></span> ';
This code removed links from your author and date .... Hope that works fine to you.
I am trying to also display the category of a related post, below one article.
I am using Newsmag theme.
The code that builds my related posts is this:
class td_module {
var $post;
var $title_attribute;
var $title;
var $href;
var $td_review; //review meta
var $category;
//constructor
function __construct($post) {
//this filter is used by td_unique_posts.php - to add unique posts to the array for the datasource
apply_filters("td_wp_boost_new_module", $post);
$this->post = $post;
$this->title = get_the_title($post->ID);
$this->title_attribute = esc_attr(strip_tags($this->title));
$this->href = esc_url(get_permalink($post->ID));
$this->category = '';
if (has_post_thumbnail($this->post->ID)) {
$this->post_has_thumb = true;
} else {
$this->post_has_thumb = false;
}
//get the review metadata
$this->td_review = get_post_meta($this->post->ID, 'td_review', true);
}
and the part that displays the related articles is this:
$buffy .= '<div class="td-module-thumb">';
if (current_user_can('edit_posts')) {
$buffy .= '<a class="td-admin-edit" href="' . get_edit_post_link($this->post->ID) . '">edit</a>';
}
$buffy .='<a href="' . $this->href . '" rel="bookmark" title="' . $this->title_attribute . '">';
$buffy .= '<img width="' . $td_temp_image_url[1] . '" height="' . $td_temp_image_url[2] . '" itemprop="image" class="entry-thumb" src="' . $td_temp_image_url[0] . '" ' . $attachment_alt . $attachment_title . '/>';
$buffy .= '<span class="td-module-thumb-category">'.$this->category.'</span>';
....................................
$this->category was added by me. I am trying to get the data from wp_terms table and display the categories of each of the related posts.
I am new to WordPress(actually, this is the first time I touch WordPress code).
Thank you
This should work with your existing code:
$buffy .= '<div class="td-module-thumb">';
$related_category = get_the_category($this->post->ID);
if (current_user_can('edit_posts')) {
$buffy .= '<a class="td-admin-edit" href="' . get_edit_post_link($this->post->ID) . '">edit</a>';
}
$buffy .='<a href="' . $this->href . '" rel="bookmark" title="' . $this->title_attribute . '">';
$buffy .= '<img width="' . $td_temp_image_url[1] . '" height="' . $td_temp_image_url[2] . '" itemprop="image" class="entry-thumb" src="' . $td_temp_image_url[0] . '" ' . $attachment_alt . $attachment_title . '/>';
$buffy .= '<span class="td-module-thumb-category">'.$related_category[0]->cat_name.'</span>';
Or if you need the category to be a link use:
$buffy .= '<div class="td-module-thumb">';
$related_category = get_the_category($this->post->ID);
if (current_user_can('edit_posts')) {
$buffy .= '<a class="td-admin-edit" href="' . get_edit_post_link($this->post->ID) . '">edit</a>';
}
$buffy .='<a href="' . $this->href . '" rel="bookmark" title="' . $this->title_attribute . '">';
$buffy .= '<img width="' . $td_temp_image_url[1] . '" height="' . $td_temp_image_url[2] . '" itemprop="image" class="entry-thumb" src="' . $td_temp_image_url[0] . '" ' . $attachment_alt . $attachment_title . '/>';
$buffy .= '<span class="td-module-thumb-category">'.$related_category[0]->cat_name.'</span>';
You will probably want to move $related_category = get_the_category($this->post->ID); to where you have $this->category = ''; in your first pasted code segment
I can't figure out how to write a ternary operator for the ending of this line (i.e. "location" and "description"). I've defined the $location and $description variables but I'm getting an error that says unexpected $location variable.
Basically, for both location and description, I'm trying to say: If the field has been filled out, display the field. If not, don't print anything.
$content .= '<div data-order="' . $count . '" data-group="' . $group . '"><div class="img-wrap"><img data-iwidth="'.$isize[0].'" data-iheight="' . $isize[1] . '" class="myslide" data-lazy="' . get_template_directory_uri() . '/img/slides/' . strtolower($group) . '/' . $image_url . '" alt="' . get_sub_field('title') . '"><div class="slide-caption"><strong>' . get_sub_field('title') . '</strong><hr>' . get_sub_field('location') ? '<em>'$location'</em>' : ''; . get_sub_field('description') ? '<hr><span class="details">Details [+]</span><div class="details-display">'$description'</div>' : ''; . '</div></div></div>';
Here is the full context:
if( have_rows('slides', $frontpage_id) ):
while ( have_rows('slides', $frontpage_id) ) :
the_row();
$group = get_sub_field('group');
$count = 1;
$i = 0;
$nav .= '<ul id="nav_'.$group.'" style="display: none;">';
$content .= '<div class="image-data-container image-container-'. $group .'">';
while( have_rows('images') ) : the_row(); $count++;
$image_url = get_sub_field('image');
$location = get_sub_field('location');
$description = get_sub_field('description');
$isize = #getimagesize(get_template_directory_uri() . '/img/slides/' . strtolower($group) . '/' . $image_url);
$content .= '<div data-order="' . $count . '" data-group="' . $group . '"><div class="img-wrap"><img data-iwidth="'.$isize[0].'" data-iheight="' . $isize[1] . '" class="myslide" data-lazy="' . get_template_directory_uri() . '/img/slides/' . strtolower($group) . '/' . $image_url . '" alt="' . get_sub_field('title') . '"><div class="slide-caption"><strong>' . get_sub_field('title') . '</strong><hr>' . get_sub_field('location') ? '<em>'$location'</em>' : ''; . get_sub_field('description') ? '<hr><span class="details">Details [+]</span><div class="details-display">'$description'</div>' : ''; . '</div></div></div>';
$nav .= '<li >' . get_sub_field('title') . '</li>';
$i++;
endwhile;
$content .= '</div>';
$nav .= '</ul>';
endwhile;
endif;
?>
Other the fact that your $content variable is a formatting nightmare. The ternary operator syntax returns result based on the condition and it should be separated from your $content attribute
$someVariable = (get_sub_field('location')) ? '<em>'.$location.'</em>' : '';
Now use that some variable inside your $content variable instead
Here is your code with comments:
while( have_rows('images') ) : the_row(); $count++;
$image_url = get_sub_field('image');
// you have already defined the location variable here
// It also means and I am assuming they your get_sub_field is returning a string
// If you are receiving a boolean or null value than and than only than you can do your ternary condition call of
// get_sub_field('location') ? '<em>'$location'</em>' : ''
// and if you are receiving a string field then you will have to explicitly check the condition
// something like this (get_sub_field('location') === '') ? '<em>'$location'</em>' : ''
// I am not that familiar with wordpress framework or the ACF plugin but it looks like get_sub_field will return something so you actually wont need the ternary condition
$location = get_sub_field('location');
// Similar case with description
$description = get_sub_field('description');
$isize = #getimagesize(get_template_directory_uri() . '/img/slides/' . strtolower($group) . '/' . $image_url);
// After removing the ternary condition your content variable will look like this
$content .= '<div data-order="' . $count . '" data-group="' . $group . '"><div class="img-wrap"><img data-iwidth="'.$isize[0].'" data-iheight="' . $isize[1] . '" class="myslide" data-lazy="' . get_template_directory_uri() . '/img/slides/' . strtolower($group) . '/' . $image_url . '" alt="' . get_sub_field('title') . '"><div class="slide-caption"><strong>' . get_sub_field('title') . '</strong><hr>' . '<em>' . $location . '</em>' . '<hr><span class="details">Details [+]</span><div class="details-display">' . $description . '</div>' . '</div></div></div>';
$nav .= '<li >' . get_sub_field('title') . '</li>';
$i++;
endwhile;
You know you can always make it more readable
// You know instead of all the concatenation you can do this instead
$templateDirURI = get_template_directory_uri();
$groupLower = strtolower($group);
$title = get_sub_field('title');
$content .= "<div data-order='{$count}' data-group='{$group}'><div class='img-wrap'><img data-iwidth='{$isize[0]}' data-iheight='{$isize[1]}' class='myslide' data-lazy='{$templateDirURI}/img/slides/{$groupLower}/{$image_url}' alt='{$title}'><div class='slide-caption'><strong>{$title}</strong><hr><em>{$location}</em><hr><span class='details'>Details [+]</span><div class='details-display'>{$description}</div></div></div></div>";
Updated Code
$content .= "<div data-order='{$count}' data-group='{$group}'><div class='img-wrap'><img data-iwidth='{$isize[0]}' data-iheight='{$isize[1]}' class='myslide' data-lazy='{$templateDirURI}/img/slides/{$groupLower}/{$image_url}' alt='{$title}'><div class='slide-caption'><strong>{$title}</strong>";
// Assuming the user did not submit location information
if($location !== "")
$content .= "<hr><em>{$location}</em><hr>";
if($description !== "")
$content .= "<span class='details'>Details [+]</span><div class='details-display'>{$description}</div>";
$content .= "</div></div></div>";