Adminimal menu including duplicate jquery.min.js file

I don't think the commit attributed to this issue is correct: http://cgit.drupalcode.org/adminimal_admin_menu/commit/?id=22bca0a1025be...

It does not contain anything remotely similar to the patch in this issue. @ANDITKO, can you check again what is going on here?

Though pointed out in #9 that the patch here wasn't actually committed in the listed commit, the patch did in fact make it in, but there's still a bug.

The problem is that even though the default should be "don't add jQuery until we need it", you have to first enable a variable to create it, and then disable it to return false. This means that on a fresh install of Adminimal Admin Menu, you will always have jQuery added twice unless you enable and disable the jQuery setting.

// line 34
// The problem is that the variables by default are set to TRUE.
// So if you never enabled/disabled 'adminimal_admin_menu_jquery', it will always be TRUE
$load_slicknav = variable_get('adminimal_admin_menu_slicknav', TRUE );
$load_jQuery = variable_get('adminimal_admin_menu_jquery', TRUE );

// Check if both slicknav and custom jQuery must be loaded.
  if ($load_slicknav and $load_jQuery) {
    drupal_add_js($path . '/js/jquery.min.js', array( 'type' => 'file', 'scope' => 'header', 'weight' => 888 ));
    drupal_add_js($path . '/js/slicknav/jquery-no-conflict.slicknav.js', array( 'type' => 'file', 'scope' => 'header', 'weight' => 888 ));
    drupal_add_css($path . '/js/slicknav/slicknav.css');
  }

Problem still not fixed, when varaibles

$load_slicknav = variable_get('adminimal_admin_menu_slicknav', TRUE );
$load_jQuery = variable_get('adminimal_admin_menu_jquery', TRUE );

enabled. And they still enabled by default. It looks like they disabled by default, but it's bug, patch to fix it #2513970-7: Don't use the string 'TRUE' as a boolean