Integrate hacked with site_audit & allow hacked to be installed globally

I can verify this works well, but I think we'll want some additional people to test it.

The patch applied cleanly and i'm able to run "drush hd views" for instance and it comes back perfectly. I'm not able to run hacked-list-projects though.

call_user_func_array() expects parameter 1 to be a valid callback, function 'hacked_build_report_batch' not found or invalid function name batch.inc:149

Any ideas?

link to issue

Cleaned up the patch and it's working for me now. Would like some other eyes on if possible.

I guess this just need some testing by me to see if it still works!

This doesn't quite work as far as I can see.

When running via drush I get the following:

$ drush hlp
Rebuilding Hacked! report
call_user_func_array() expects parameter 1 to be a valid callback, function 'hacked_build_report_batch' not found or invalid function    [warning]
name batch.inc:149
call_user_func_array() expects parameter 1 to be a valid callback, function 'hacked_build_report_batch' not found or invalid function    [warning]
name batch.inc:149
usort() expects parameter 1 to be array, null given hacked.drush.inc:124                                                                 [warning]
Done.
Invalid argument supplied for foreach() hacked.drush.inc:154                                                                             [warning]
 Title  Name  Version  Status  Changed  Deleted 
                                                

If I place the hacked module into the Drupal codebase, then it does work, but if the module is installed as a Drush extension, then it doesn't.

I was testing https://drupal.org/files/issues/hacked-standalone_drush-2066371-4_0.patch

FluxSauce, are you able to give me some more details on "that patch really didn't work for me at all, and it removed out a lot of logic that was used to get it working in the first place place."?

Did the patch not apply, or did the drush hacked command not work. Did it produce any errors?

Ok, maybe I got it. The file argument for the batch operation wasn't working. It was prepending the root of the site you call the command from to the root of the hacked drush commands directory, eg. /var/www/sitename//usr/share/php/drush...). What I've done for now is removed the file argument and made a _drush copy of hacked_build_report_batch and put that directly into the hacked.drush.inc file.Most of the other modifications are from FluxSauce's previous patch, but redone against the latest from hacked's git.

It would be great if someone could test this one out.

Still applies to 7.x-2.x but need to get rid of: drush_log(print_r($operations), 'warning');

Sorry about that. Here's a new patch with the line removed as well as an interdiff.

Unfortunately, if you've got a site that already has hacked in it's module base, you'll get a fatal with this. Anyone have any idea how to get around that?

Would it make sense to just remove it from the site?

I'm getting:

"Unable to determine whether core or contrib modules have been modified!"

hacked-list-projects doesn't have the --include-unchanged=0 argument.

If I add --strict=0 to line #79 of hacked.site_audit.inc I get a list of hacked projects in my report without the unchanged modules listed.

$result = drush_invoke_process('@self', 'hacked-list-projects', array(), array('--include-unchanged=0', '--strict=0'), FALSE);

I just used the patch from #20 against a git clone, aka the dev branch, and installed the resulting patched files into my home/.drush directory, cleared caches, and used that to
a) try the hacked (hd) command and it found the changes I expected (we have a patched version of core to support our proxy for https traffic)
b) use the site audit (aa) command, although while it executed hacked (I presume) I got the result "Unable to determine whether core or contrib modules have been modified!" Not sure why that happened, it appears that #20 incorporates the --strict=0 from #19, although I found that at line 79 of hacked.site_audit.inc and not hacked.drush.inc, which I assume was a typo in the comment above.

So I tried just "drush hacked-list-projects" and got the error message
PHP Fatal error: Cannot redeclare hacked_reports_hacked() (previously declared in /sites/all/modules/_development_d7/hacked/hacked.report.inc:9) in .drush/hacked/hacked.report.inc on line 21 which is documented in #14. I removed the hacked module from the site's module directory and the hacked-list-projects and site-audit commands worked fine. Is there no way to test if a module is installed in drush?

I failed to apply patch to latest dev release:

hacked.info:

; Information added by Drupal.org packaging script on 2015-02-19
version = "7.x-2.0-beta5+13-dev"
core = "7.x"
project = "hacked"
datestamp = "1424333282"

Applying patch:

$ patch -p1 integrate_hacked_with-2066371-20.patch 
patching file hacked.drush.inc
Hunk #3 FAILED at 86.
1 out of 16 hunks FAILED -- saving rejects to file hacked.drush.inc.rej
patching file hacked.module
patching file hacked.report.inc
patching file hacked.site_audit.inc
can't find file to patch at input line 617
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/includes/hacked_project.inc b/includes/hacked_project.inc
|index 11c44fe..4cc3b7a 100644
|--- a/includes/hacked_project.inc
|+++ b/includes/hacked_project.inc
--------------------------
File to patch: 

Is patch incompatible with latest dev release?

This applies cleanly to commit 1fa55b3

The removal of includes/hacked_project.inc in commit 794c51e (and other code rearranging) is what is causing the conflict.

Assuming we can reroll the patch to work, can the maintainer comment about what is holding back this patch from being committed?

Patch #26 works for me.. Please add this functionality.

looks good to me.

Looks like this is now breaking when run from the UI, over at admin/reports/hacked:

An AJAX HTTP error occurred. HTTP Result Code: 200 Debugging information follows. Path: /batch?id=13366&op=do StatusText: OK ResponseText: ( ! ) Fatal error: Call to undefined function hacked_load_dependencies() in /path/to/drupal/sites/all/modules/contrib/hacked/hacked.report.inc on line 41 Call Stack #TimeMemoryFunctionLocation 10.0001243520{main}( )../index.php:0 20.07149387544menu_execute_active_handler( )../index.php:21 30.07169389072call_user_func_array:{/path/to/drupal/includes/menu.inc:519} ( )../menu.inc:519 40.07169389304system_batch_page( )../menu.inc:519 50.07169389408_batch_page( )../system.admin.inc:2379 60.07169390264_batch_do( )../batch.inc:80 70.07169390480_batch_process( )../batch.inc:161 80.07229423720call_user_func_array:{/path/to/drupal/includes/batch.inc:284} ( )../batch.inc:284 90.07229423832hacked_build_report_batch( )../batch.inc:284

I'm assuming hacked.drush.inc only gets loaded when run through drush? That's where that function is defined.

I thought it would make more sense to get this working in D8 first. The problem, however, is that I'm running into this:

exception 'Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException' with message 'You have requested a non-existent service "cache.hacked".' in /path/to/drupal/core/lib/Drupal/Component/DependencyInjection/Container.php:161 [error]

Stack trace:

• #0 /path/to/drupal/template/core/lib/Drupal.php(299): Drupal\Component\DependencyInjection\Container->get('cache.hacked')
• #1 /path/to/hacked/hacked.drush.inc(84): Drupal::cache('hacked')

I think the problem is that the Hacked! cache service cache.hacked defined in hacked.services.yml was never set up because the module wasn't installed. Anyone know if there's a way to get to Drush to do so, without installing the module?

services:
  cache.hacked:
    class: Drupal\Core\Cache\CacheBackendInterface
    tags:
      - { name: cache.bin }
    factory: cache_factory:get
    arguments: [hacked]

Maybe the service can be registered via ServiceProviderBase::register as described in Overriding services in Drupal 8 - advanced cases, but I'm not sure.

Here's a D7 version that should be good to go; it works both uninstalled with Drush and installed from the Web UI now. Changes from #26:

Added the following stanza to hacked.report.inc:

// Add missing dependencies if the module isn't installed.
if (function_exists('drush_verify_cli') && drush_verify_cli()) {
  hacked_load_drush_dependencies();
}

...and renamed hacked_load_dependencies() to hacked_load_drush_dependencies().

Leaving status as Needs Work for D8, even though this D7 patch should really be Needs Review.

Don't worry about #30. I just realized I was trying to use the Drupal cache instead of Drush's cache. Fixing...

Well yeah to be clear about the problem described earlier. Services aren't loaded into the container, unless the module is installed. Maybe drush could provide its own dependency injection container, but I guess this could be tricky.

I didn't get a chance to test this with Site Audit (even though the code is there), but I tested the following:

• Running via Drush without module installed.
• Running via Drush with module installed.
• Running from the Web UI (module is installed obviously).

Latest 7.x-2.x-dev (3 Jul 2017) tarball was crashing using drush with the message

$ drush hlp
..... (cut)
Finished processing: Zen

Done.
Invalid argument supplied for foreach() hacked.drush.inc:155

 Title  Name  Version  Status  Changed  Deleted

[Edit]
The patch from #26 resolved the drush issues, but the Drupal Admin UI functionality was fairly broken.

#35: For D7, #31 was the way to go, which I just committed.

In D7, this seems to cause problems when used with #1835444: Support the new dev info strings. See new issue #3019685: Class 'hackedProjectWebDevDownloader' not found in hackedProject->identify_project() for details.

We need to update the README to include the new functionality here, as was done for D7.