Refactoring jboss module to work with the Mixin

Removed datastore USERNAME and PASSWORD which are provided by
Msf::Exploit::Remote::HttpClient. Removed datastore PATH and VERB which
are provided by the mixin (lib/msf/http/jboss). Moved target detection
to the mixin.
Refactoring jboss module to work with the Mixin

Moved upload and delete methods of deploymentfilerepository to the
mixin. Removed call_uri_mtimes method as the module now uses deploy
from the mixin.
Jboss file deployment repository refactorization

Moved lib/msf/http/jboss/bean_shell_script.rb to
lib/msf/http/jboss/script.rb. Moved head_stager_jsp to script.rb.
Removed stager_jsp to use the function from the mixin.
Refactor spec to match lib/msf/jboss

Moved bean_hsell_scripts_spec to scripts_spec. Fixed a typo.
Fix problem with HEAD requests

Split lib/msf/http/jboss/script into
lib/msf/http/jboss/deployment_file_repository_scripts.rb and
lib/msf/http/jboss/bean_shell_scripts.rb as
Fixed spec for lib/msf/http/jboss

Revert commit abdd72e8c66f9da8864022adf1d489f07020356e.
Added some spec for lib/msf/http/jboss/deployment_file_repository_scripts
Fix timeout with HEAD request in delete_file
Refactor auto_target from Jboss mixin

Removed fail_with and targets from the mixin.
Refactor Jboss mixin

Use send_request_cgi and vars_get
rand_text_alpha -> Rex::Text.rand_text_alpha
Added YARD doc to the mixin

Also make a slight correction on jboss_deployementfilerepository.rb to
handle nil responses.
Added spec to new methods in Jboss mixin
Slight modifications to pass msftidy
Added spec for jboss/deployment_file_repository
Fix a typo that was preventing WAR deletion

I made a typo while refactoring jboss_deploymentfilerepository. This
typo was preventing the WAR payload to be removed after its execution.
Spec modification to allow previous commit

The spec was requiring the WAR to be at /deploy but this path was a
typo introduce while refactoring. The initial correct path is
/deploy/management
Use vars_get instead of direct URI concatenation
Squashed commit of the following:

commit 7f659547b3b5d8de34a1e4b136b5e899ae7878da
Author: Tod Beardsley <todb@metasploit.com>
Date:   Mon Dec 1 07:54:53 2014 -0600

    Add a pointer to true and correct docs

    See #4000. Seems kind of pointless to ship all these bytes when we host
    everything on GitHub and provide tools to generate docs yourself.

commit 01668c8529454ffb0438aadef62eafb5045503ce
Author: Tod Beardsley <todb@metasploit.com>
Date:   Mon Dec 1 07:39:11 2014 -0600

    Remove old documentation

    The last significant commit in this directory was a844b5c3 , which
    touched a sample browser module. Before that, changes were either
    ranging across the codebase, or surgical removal of out of date
    documentation.

    This all is looking to me like fairly useless historical artifacts.

    See #4000 for more discussion.
Fail if init_module_paths called more than once

MSP-11672

Calling init_module_paths takes 6 seconds on my machine even when there are no
files to that are changed just because it takes that long to walk the
directories and gather the mtime for each file.  Therefore, calling it
more than once should be avoided.  Also, there is no reason to call it
twice as to add paths later, `modules.add_module_paths` should be used.
init_module_paths once in msfconsole

MSP-11672

Pass `'DeferModuleLoads' => false` to `Msf::Simple::Framework.create` so
that `framework.modules.init_module_paths` is only called once (directly
in `Msf::Ui::Console::Driver#initialize`) instead of twice (in
`Msf::Simple::Framework.create` and `Msf::Ui::Console::Driver#initialize).
Merge branch 'master' into bug/MSP-11672/double-init-module-paths

MSP-11672