podlove / podlove-subscribe-button-wp-plugin

js/admin.js

(function($) {

    var entityMap = {
      '&': '&',
      '<': '&lt;',
      '>': '&gt;',
      '"': '&quot;',
      "'": '&#39;',
      '/': '&#x2F;',
      '`': '&#x60;',
      '=': '&#x3D;'
    };
    
    function escapeHtml(string) {
      return String(string).replace(/[&<>"'`=\/]/g, function (s) {
        return entityMap[s];
      });
    }

	function podlove_init_color_buttons() {
        jQuery(document).ready( function($) {
            var params = {
                change: function(e, ui) {
                    $( e.target ).val( ui.color.toString() );
                    $( e.target ).trigger('change'); // enable widget "Save" button
                },
            }

            $('.podlove_subscribe_button_color').not('[id*="__i__"]').wpColorPicker( params );
        })
	}

	$( document ).ready( function() {

		$("#Podlove_cover_image_select").on( 'click', function(event) {

The parameter event is not used and could be removed.

			podlove_cover_image_selector = wp.media.frames.customHeader = wp.media( {

The variable wp seems to be never declared. If this is a global, consider adding a /** global: wp */ comment.

The variable podlove_cover_image_selector seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.podlove_cover_image_selector.

					title: i18n.media_library,

The variable i18n seems to be never declared. If this is a global, consider adding a /** global: i18n */ comment.

					library: {
						type: 'image'
					},
					button: {
						text: i18n.use_for
			   		},
			   		multiple: false
			    } );
			podlove_cover_image_selector.open();

			podlove_cover_image_selector.on('select', function() {
				var podcast_image_url = podlove_cover_image_selector.state().get('selection').first().toJSON().url;
				$("#podlove-button-cover").val(podcast_image_url);
				$("#podlove-button-cover").trigger('change');
			});
		} );

		$(document).ready(function () {
		    podlove_init_color_buttons();

		    jQuery(document).on('widget-updated', podlove_init_color_buttons);
		    jQuery(document).on('widget-added', podlove_init_color_buttons);

		    // re-init after saving configs
		    jQuery(document).on('ajaxComplete', function(e){

The parameter e is not used and could be removed.

		        podlove_init_color_buttons();
		    });
		})

		var feed_counter = 0;
		var source = $("#feed_line_template").html();

		$(".add_feed").on( 'click', function () {
			add_new_feed();
		} );

		if ( window.feeds !== undefined ) {
			$.each( feeds, function (index, feed) {

The variable feeds seems to be never declared. If this is a global, consider adding a /** global: feeds */ comment.

				add_existing_feed(feed);
			} );
		}

		function add_new_feed() {
			row = source.replace( /\{\{url\}\}/g, '' );

The variable row seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.row.

			row = row.replace( /\{\{itunesfeedid\}\}/g, '' );
			row = row.replace( /\{\{id\}\}/g, feed_counter );

			$("#feeds_table_body").append(row);

			new_row = $("#feeds_table_body tr:last");

The variable new_row seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.new_row.

			new_row.find("input:first").focus();

			$(".podlove-icon-remove").on( 'click', function () {
				$(this).closest("tr").remove();
			} );

			feed_counter++;
		}

		function add_existing_feed( feed ) {
			row = source.replace( /\{\{url\}\}/g, escapeHtml(feed.url) );

The variable row seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.row.

			row = row.replace( /\{\{id\}\}/g, feed_counter );
			if ( feed.itunesfeedid == null ) {

Comparing feed.itunesfeedid to null using the == operator is not safe. Consider using === instead.

				row = row.replace( /\{\{itunesfeedid\}\}/g, '' );
			} else {
				row = row.replace( /\{\{itunesfeedid\}\}/g, escapeHtml(feed.itunesfeedid) );
			}

			$("#feeds_table_body").append(row);

			new_row = $("#feeds_table_body tr:last");

The variable new_row seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.new_row.

			new_row.find('select.podlove-media-format option[value="' + escapeHtml(feed.format) + '"]').attr('selected',true);

			$(".podlove-icon-remove").on( 'click', function () {
				$(this).closest("tr").remove();
			} );

			feed_counter++;
		}

	} );
}(jQuery));