testimonials.js ➔ reloadCarouselTestimonial - Code Metrics - Inspection of "Merge branch 'master' of https://github.com/davide..." - davide-casiraghi/laravel-testimonials - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c4765e...785c00 )

by Davide

created 2019-07-19 08:53 UTC

testimonials.js ➔ reloadCarouselTestimonial A

↳ Parent: resources/assets/js/testimonials.js

Complexity

Conditions

Size

Total Lines	5
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	4
c	0
b	0
f	0
dl	0
loc	5
rs	10
cc	1



// Create carousel 
    function createCarouselTestimonial(numberOfSlides){
        jQuery('.testimonialsList').not('.slick-initialized').slick({
            dots: true,
            arrows: true,
            infinite: true,
            slidesToShow: numberOfSlides,
            slidesToScroll: 1,
            autoplay: true,
  			autoplaySpeed: 6000,
  			pauseOnHover: true
        });
	}

// Calculate number of slides to show
	function calculateNumberOfSlidesToShowTestimonial(){
		var carouselWidth = jQuery(".testimonialsList").width();
		var numberOfSlides = 0;
      	switch (true) {
		    case (carouselWidth < 767):
		        numberOfSlides = 1;
		        break;
		    case (carouselWidth < 991):
		        numberOfSlides = 2;
		        break;
		    case (carouselWidth < 1199):
		        numberOfSlides = 3;
		        break;
		    case (carouselWidth > 1200):
		        numberOfSlides = 3;
		        break;
		}

		return numberOfSlides;
	}

// Reload Carousel on browser resize (to make it responsible)

    function reloadCarouselTestimonial () {
      	jQuery('.testimonialsList').slick('unslick');
      	numberOfSlides = calculateNumberOfSlidesToShowTestimonial();

	    createCarouselTestimonial(numberOfSlides);
    }

// Call updateMaxHeight when browser resize event fires
    jQuery(window).on("resize", reloadCarouselTestimonial);



jQuery(document).ready(function () {
	// Start carousel
	    if (jQuery(".testimonialsList").length) {
	        setTimeout(function () {
	        	numberOfSlides = calculateNumberOfSlidesToShowTestimonial();

	            createCarouselTestimonial(numberOfSlides);
	        }, 300);
	    }


});


1
2
3			// Create carousel
4			function createCarouselTestimonial(numberOfSlides){
5			jQuery('.testimonialsList').not('.slick-initialized').slick({
6			dots: true,
7			arrows: true,
8			infinite: true,
9			slidesToShow: numberOfSlides,
10			slidesToScroll: 1,
11			autoplay: true,
12			autoplaySpeed: 6000,
13			pauseOnHover: true
14			});
15			}
16
17			// Calculate number of slides to show
18			function calculateNumberOfSlidesToShowTestimonial(){
19			var carouselWidth = jQuery(".testimonialsList").width();
20			var numberOfSlides = 0;
21			switch (true) {
22			case (carouselWidth < 767):
23			numberOfSlides = 1;
24			break;
25			case (carouselWidth < 991):
26			numberOfSlides = 2;
27			break;
28			case (carouselWidth < 1199):
29			numberOfSlides = 3;
30			break;
31			case (carouselWidth > 1200):
32			numberOfSlides = 3;
33			break;
34			}
35
36			return numberOfSlides;
37			}
38
39			// Reload Carousel on browser resize (to make it responsible)
40
41			function reloadCarouselTestimonial () {
42			jQuery('.testimonialsList').slick('unslick');
43			numberOfSlides = calculateNumberOfSlidesToShowTestimonial();
			0 ignored issues – show Bug introduced 2019-07-19 09:08 UTC by Report Bug Copy Issue Report The variable `numberOfSlides` 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.numberOfSlides`. Loading history...
44			createCarouselTestimonial(numberOfSlides);
45			}
46
47			// Call updateMaxHeight when browser resize event fires
48			jQuery(window).on("resize", reloadCarouselTestimonial);
49
50
51
52			jQuery(document).ready(function () {
53			// Start carousel
54			if (jQuery(".testimonialsList").length) {
55			setTimeout(function () {
56			numberOfSlides = calculateNumberOfSlidesToShowTestimonial();
			0 ignored issues – show Bug introduced 2019-07-19 09:08 UTC by Report Bug Copy Issue Report The variable `numberOfSlides` 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.numberOfSlides`. Loading history...
57			createCarouselTestimonial(numberOfSlides);
58			}, 300);
59			}
60
61
62			});
63

davide-casiraghi / laravel-testimonials

Push — master ( c4765e...785c00 )

testimonials.js ➔ reloadCarouselTestimonial A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like