На что ругается консоль?

Question

[Чёкатилло]

Что ей надо не пойму, вроде все на месте здесь

Если проскроллить вниз, подгрузятся превью, и в этот момент в консоли ругань на строку:

$container.append(result).masonry('appended', result);

Использую

к (весь) нему скрипт:

$(function () {

  $('.loop-loader').fadeIn(100);
var $boxes = $('.loop');
	$boxes.hide();

  var $container = $('#content_masonry');
  $container.imagesLoaded( function() {
            $boxes.fadeIn(100);
  $('.loop-loader').fadeOut(100);

    $container.masonry({
        itemSelector : '.loop',
        columnwidth: 300,
        gutter: 24,
        isFitWidth: true,
        isOriginLeft: true,
        isAnimated: !Modernizr.csstransforms
    });
  });


var msnry = $container.data('masonry');
$container.infiniteScroll({
  path: '#page-nav a',
  append: '.loop',
  outlayer: msnry,
  history: false
});
});

	var loading = false;
	$(window).scroll(function () {
		if (!loading && $(window).scrollTop() + $(window).height() > $(document).height() - 300) {
			loading = true;
			loadContents();
		}
	});

	function loadContents() {
		var url = $('#page-nav a').attr('href');
		if (url) {
    $(".content-loader").show(100);
			$.ajax({
				type: "GET",
				url: url,
				dataType: "html",
				success: function (loaded) {
					var result = $(loaded).find('#content_masonry .loop');
					var nextlink = $(loaded).find('#page-nav a').attr('href');
					$(loaded).imagesLoaded(function() {
						$container.append(result).masonry('appended', result);
						$('.popup-post-modal-open').bind('click', DimaPopup);

					});

    $(".content-loader").hide(100);
					if (nextlink != 'undefined') {
						$('#page-nav a').attr('href', nextlink);
					} else {
						$('#page-nav').remove();
					}
					loading = false;
				}
			});
		}
	}

Подскажите пожалуйста, что не так?
Спасибо!

Comments

[Сергей delphinpro]

Ну а где сама ругань?

PS текст ошибки где, если непонятно

[Чёкатилло]

Сергей, ошибка пишет такая: Uncaught ReferenceError: $container is not defined

[Сергей delphinpro]

Чёкатилло,
ругается на то, что не определена переменная $container

Почему она не определена, я мог бы вам сказать, если бы вы выложили аккуратно отформатированный код с отступами, и т.д. (Да вы бы и сами поняли по аккуратному коду).

А копаться в этой простыне — увольте.

[Чёкатилло]

Сергей, типа так? =)

$(function() {
    $('.loop-loader').fadeIn(
        100);
    var $boxes = $('.loop');
    $boxes.hide();
    var $container = $(
        '#content_masonry');
    $container.imagesLoaded(
        function() {
            $boxes.fadeIn(
                100);
            $(
                '.loop-loader'
            ).fadeOut(
                100);
            $container.masonry({
                itemSelector: '.loop',
                columnwidth: 300,
                gutter: 24,
                isFitWidth: true,
                isOriginLeft: true,
                isAnimated:
                    !
                    Modernizr
                    .csstransforms
            });
        });
    var msnry = $container.data(
        'masonry');
    $container.infiniteScroll({
        path: '#page-nav a',
        append: '.loop',
        outlayer: msnry,
        history: false
    });
});
var loading = false;
$(window).scroll(function() {
    if (!loading && $(window).scrollTop() +
        $(window).height() > $(
            document).height() -
        300) {
        loading = true;
        loadContents();
    }
});

function loadContents() {
    var url = $('#page-nav a').attr(
        'href');
    if (url) {
        $(".content-loader").show(100);
        $.ajax({
            type: "GET",
            url: url,
            dataType: "html",
            success: function(
                loaded) {
                var result =
                    $(
                        loaded
                    ).find(
                        '#content_masonry .loop'
                    );
                var nextlink =
                    $(
                        loaded
                    ).find(
                        '#page-nav a'
                    ).attr(
                        'href'
                    );
                $(loaded).imagesLoaded(
                    function() {
                        $container
                            .append(
                                result
                            )
                            .masonry(
                                'appended',
                                result
                            );
                        $
                            (
                                '.popup-post-modal-open'
                            )
                            .bind(
                                'click',
                                DimaPopup
                            );
                    });
                $(
                    ".content-loader"
                ).hide(
                    100
                );
                if (
                    nextlink !=
                    'undefined'
                ) {
                    $(
                        '#page-nav a'
                    ).attr(
                        'href',
                        nextlink
                    );
                } else {
                    $(
                        '#page-nav'
                    ).remove();
                }
                loading =
                    false;
            }
        });
    }
}

[Сергей delphinpro]

Чёкатилло, Почти так. Перебор, конечно, но даже тут видно, что переменная $container объявлена в одном скоупе, а используется в другом.

PS Форматировать лучше так

spoiler

$(function() {

  $('.loop-loader').fadeIn(100);
  var $boxes = $('.loop');
  $boxes.hide();

  var $container = $('#content_masonry');
  $container.imagesLoaded(function() {
    $boxes.fadeIn(100);
    $('.loop-loader').fadeOut(100);

    $container.masonry({
      itemSelector: '.loop',
      columnwidth : 300,
      gutter      : 24,
      isFitWidth  : true,
      isOriginLeft: true,
      isAnimated  : !Modernizr.csstransforms
    });
  });

  var msnry = $container.data('masonry');
  $container.infiniteScroll({
    path    : '#page-nav a',
    append  : '.loop',
    outlayer: msnry,
    history : false
  });
});

var loading = false;
$(window).scroll(function() {
  if (!loading && $(window).scrollTop() + $(window).height() > $(document).height() - 300) {
    loading = true;
    loadContents();
  }
});

function loadContents() {
  var url = $('#page-nav a').attr('href');
  if (url) {
    $('.content-loader').show(100);
    $.ajax({
      type    : 'GET',
      url     : url,
      dataType: 'html',
      success : function(loaded) {
        var result   = $(loaded).find('#content_masonry .loop');
        var nextlink = $(loaded).find('#page-nav a').attr('href');
        $(loaded).imagesLoaded(function() {
          $container.append(result).masonry('appended', result);
          $('.popup-post-modal-open').bind('click', DimaPopup);

        });

        $('.content-loader').hide(100);
        if (nextlink != 'undefined') {
          $('#page-nav a').attr('href', nextlink);
        } else {
          $('#page-nav').remove();
        }
        loading = false;
      }
    });
  }
}