This patch does the bare minimum to get the module working in Drupal 7. Not complete in the least, but a place to start. Did this here in Stockholm at the documentation sprint so Jam can start jammin' on his idea for a next-gen admin help system.

CommentFileSizeAuthor
#3 advanced_help_with_search.patch18.44 KBjoshk
#1 advanced_help_d7.patch13.24 KBjoshk

Comments

joshk’s picture

joshk commented
StatusFileSize
new advanced_help_d7.patch13.24 KB

Here's a much better patch. Fixes theme function/call and popups. The example module works as expected!

joshk’s picture

joshk commented

Ok, fair warning: search doesn't work. I'm not sure I understand how it was using do_search before, and now we don't have that, soooo....

http://drupal.org/update/modules/6/7#search-api

More work for later!

joshk’s picture

joshk commented
StatusFileSize
new advanced_help_with_search.patch18.44 KB

Hey hey hey! Now with working search.

horncologne’s picture

horncologne commented 
+++ advanced_help.install	14 Nov 2009 16:57:29 -0000
@@ -16,6 +16,19 @@ function advanced_help_uninstall() {
+ * Implementation of hook_enable.

advanced_help_enable() ends up in the file twice after patching.

Enabling the modules gives this warning:

Warning: in_array(): Wrong datatype for second argument in advanced_help_enable() (line 25 of /Users/jam/Sites/drupal-7.x-dev/sites/all/modules/advanced_help/advanced_help.install).

I'm on crack. Are you, too?

horncologne’s picture

horncologne commented

I also had to change the Drupal version in the .info files.

I'm not sure if the above is due to my patching technique or a problem with the patch.

robloach’s picture

robloach
he/him
commented

Advanced Help is a wicked system, and I'd definitely love for it to be slick in Drupal 7. Hence this marks my subscription post and my note of personal brainstorming of how it could improve on the slick/JavaScript-y side. Thanks!

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Needs work » Fixed

Porting was actually done in the meantime. If your port had additional features over the existing (straight) port, could you point them out, and possiby reroll against the D7.dev version ?

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.