Changeset 2086040

Timestamp:

05/11/2019 10:09:05 PM (7 years ago)

Author:

Znuff

Message:

Fixed some small issues

Location:

wp-imagelightbox/trunk

Files:

• readme.txt (modified) (1 diff)
• wp-imagelightbox.php (modified) (4 diffs)

wp-imagelightbox/trunk/readme.txt

@@ -39 +39 @@
 
 == Changelog ==
+= r8 =
+* Fixed an issue with the init script being present on archive pages
+* Remove the $type from the data-imagelightbox attribute, as we don't use that anymore
 
 = r7 = 

wp-imagelightbox/trunk/wp-imagelightbox.php

@@ -4 +4 @@
  * Plugin URI: https://linge-ma.ro/wp-imagelightbox
  * Description: Responsive and Touch-Friendly Lightbox for Wordpress, JS by Osvaldas Valutis.
- * Version: r7
+ * Version: r8
  * Author: Znuff
  * Author URI: https://linge-ma.ro
@@ -11 +11 @@
 
 function iljs_enqueue() {
-   wp_enqueue_style('imagelightbox', plugin_dir_url(__FILE__).'imagelightbox.css', false, 'r6');
-   wp_enqueue_script('imagelightbox', plugin_dir_url(__FILE__).'imagelightbox.js', array('jquery'), 'r6', true);
+   wp_enqueue_style('imagelightbox', plugin_dir_url(__FILE__).'imagelightbox.css', false, 'r8');
+   wp_enqueue_script('imagelightbox', plugin_dir_url(__FILE__).'imagelightbox.js', array('jquery'), 'r8', true);
 }
 
@@ -18 +18 @@
 function iljs_mod_tags ($content) {
    global $post;
-   $type="";
 
    $pattern = "/<a(.*?)href=('|\")(.*?).(bmp|gif|jpeg|jpg|png)('|\")(.*?)>/i";
-   $replacement = '<a$1href=$2$3.$4$5$6 data-imagelightbox="'.$type.'">';
+   $replacement = '<a$1href=$2$3.$4$5$6 data-imagelightbox>';
    $content = preg_replace($pattern, $replacement, $content);
 
@@ -61 +60 @@
       // adds the filter for image galleries
       add_filter('wp_get_attachment_link','iljs_mod_tags');
+
+      // add init script
+      add_action('wp_footer', 'iljs_add_trigger', 9999);
    }
 }
 
 add_filter("template_redirect", "iljs_mod_content", 10, 1);
-add_action('wp_footer', 'iljs_add_trigger', 9999);
 ?>