File chessboard-0.3.0.js has 1067 lines of code (exceeds 275 allowed). Consider refactoring.

Open

/*!
 * chessboard.js v0.3.0
 *
 * Copyright 2013 Chris Oakman
 * Released under the MIT license

Similar blocks of code found in 2 locations. Consider refactoring.

Open

function mouseleaveSquare(e) {
  // do not fire this event if we are dragging a piece
  // NOTE: this should never happen, but it's a safeguard
  if (DRAGGING_A_PIECE !== false) return;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

function mouseenterSquare(e) {
  // do not fire this event if we are dragging a piece
  // NOTE: this should never happen, but it's a safeguard
  if (DRAGGING_A_PIECE !== false) return;

• Read up
  Read up

Function queens_under_attack has a Cognitive Complexity of 25 (exceeds 11 allowed). Consider refactoring.

Open

function queens_under_attack($board)
{
    global $attacked; // for display outside of this function
    if (!is_array($board)) {
        return 'ERROR';

• Read up
  Read up

Function expandConfig has 64 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function expandConfig() {
  if (typeof cfg === 'string' || validPositionObject(cfg) === true) {
    cfg = {
      position: cfg
    };

Function calculateAnimations has 46 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function calculateAnimations(pos1, pos2) {
  // make copies of both
  pos1 = deepCopy(pos1);
  pos2 = deepCopy(pos2);

Function buildBoard has 44 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function buildBoard(orientation) {
  if (orientation !== 'black') {
    orientation = 'white';
  }

Function checkDeps has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function checkDeps() {
  // if containerId is a string, it must be the ID of a DOM node
  if (typeof containerElOrId === 'string') {
    // cannot be empty
    if (containerElOrId === '') {

Method queens_under_attack has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function queens_under_attack($board)
{
    global $attacked; // for display outside of this function
    if (!is_array($board)) {
        return 'ERROR';

Function stopDraggedPiece has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function stopDraggedPiece(location) {
  // determine what the action should be
  var action = 'drop';
  if (location === 'offboard' && cfg.dropOffBoard === 'snapback') {
    action = 'snapback';

Function doAnimations has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function doAnimations(a, oldPos, newPos) {
  ANIMATION_HAPPENING = true;

  var numFinished = 0;
  function onFinish() {

Avoid using undefined variables such as '$result' which will lead to PHP notices.

Open

        $result[$name] = $value + $arrayB[$name];

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$result' which will lead to PHP notices.

Open

    return $result;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$tmp' which will lead to PHP notices.

Open

        $tmp[strlen(decbin($row[$checkRow])) - 1] = 1 << ($board - $checkRow - 1);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Method findRotation has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function findRotation($row, $board, $solutions)
{
    $tmp = rotateBoard($row, $board);  // Rotated 90
    if (!in_array($tmp, $solutions)) {
        $solutions[] = $tmp;

Function position has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

widget.position = function(position, useAnimation) {
  // no arguments, return the current position
  if (arguments.length === 0) {
    return deepCopy(CURRENT_POSITION);
  }

Function objToFen has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function objToFen(obj) {
  if (validPositionObject(obj) !== true) {
    return false;
  }

Function animateSquareToSquare has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function animateSquareToSquare(src, dest, piece, completeFn) {
  // get information about the source and destination squares
  var srcSquareEl = $('#' + SQUARE_ELS_IDS[src]);
  var srcSquarePosition = srcSquareEl.offset();
  var destSquareEl = $('#' + SQUARE_ELS_IDS[dest]);

Function beginDraggingPiece has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function beginDraggingPiece(source, piece, x, y) {
  // run their custom onDragStart function
  // their custom onDragStart function can cancel drag start
  if (typeof cfg.onDragStart === 'function' &&
      cfg.onDragStart(source, piece,

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 33 and the first side effect is on line 3.

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check