hello

a drupal 7 port would be nice :P

CommentFileSizeAuthor
#23 d7_port_908926_23.patch77.57 KBjasonsavino
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented
Status: Active » Postponed

I'll be happy to entertain a patch.

mlncn’s picture

mlncn CreditAttribution: mlncn commented
Issue tags: +dgd7

Subscribing. I'm entertaining implementing the Developer tool - technical sitemap idea. I think it's different enough for it to be its own module (agree/disagree?), but i would like to provide an implementation of hook_sitedoc to a Drupal 7 version of sitedoc. The sort of reports requested in this tool are pretty much full page so not sure how feasible that is; maybe if sitedoc is installed to suppress this module's menu in favor of living under sitedoc's.

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented
Title: D7 port » Sitedoc: D7 port

I was considering a hook_sitedoc so that other modules could add stuff to the sitedoc report. Much of what Zoe is thinking of would be such an addition but doesn't really belong in the base version, although some of the basis for generating that information is available.

But I can tell you that you are likely to run out of memory doing some of this.

alarfaj’s picture

alarfaj CreditAttribution: alarfaj commented

Subscribing

Tebb’s picture

Tebb CreditAttribution: Tebb commented

Subscribing.

jeffwidman’s picture

jeffwidman CreditAttribution: jeffwidman commented

subscribe

yrvyn’s picture

yrvyn CreditAttribution: yrvyn commented

subscribe

Tsubo’s picture

Tsubo CreditAttribution: Tsubo commented

subscribe

dwalker51’s picture

dwalker51 CreditAttribution: dwalker51 commented

Subscribe

mraichelson’s picture

mraichelson CreditAttribution: mraichelson commented

subscribe

grguth’s picture

grguth CreditAttribution: grguth commented

subscribe

Clean’s picture

Clean CreditAttribution: Clean commented

subscribe

flightrisk’s picture

flightrisk CreditAttribution: flightrisk commented

No news yet on a D7 version??

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented

Not yet

rolandk’s picture

rolandk CreditAttribution: rolandk commented

+1

ambereyes’s picture

ambereyes CreditAttribution: ambereyes commented

subscribe

jmones’s picture

jmones CreditAttribution: jmones commented

subscribe

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented

Stop subscribing, start following

Tsubo’s picture

Tsubo CreditAttribution: Tsubo commented

Will following help? ;)

burt.lo’s picture

burt.lo CreditAttribution: burt.lo commented

Hey, how about breaking up the module into smaller bits? That way we could choose which parts we want documented. Maybe that would make a D7 version more easily attainable, and we could at least get the easier portioned cranked out sooner than later.

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented

I'd be interested in a patch.

Frank Ralf’s picture

Frank Ralf CreditAttribution: Frank Ralf commented

I've seen that there's already a D7 branch on Git at http://drupal.org/node/145480/git-instructions/7.x-1.x

What's the status of that one?

jasonsavino’s picture

jasonsavino CreditAttribution: jasonsavino commented
FileSize
d7_port_908926_23.patch77.57 KB

Here is a patch that gets the module up and running. I am able to run the report for each of the options without error.
This is in no way a complete port. There are a few minor things that need to be fixed. ex. the show content links on tables need fixing
I am going to work some more to finish it up but thought I would give what I have so far.

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented

Thanks, Jason. Welcome to the team.

yannickoo’s picture

yannickoo
Location Berlin
CreditAttribution: yannickoo commented

BTW a 7.x-1.x branch exsists.

ChristophWeber’s picture

ChristophWeber CreditAttribution: ChristophWeber commented

I can confirm that Jason's patch from #23 applied against HEAD of the 7.x-1.x branch works for the most part.
Report known to break (WSOD): URL aliases
There is a fair amount of red ink of the type, but the report comes back fine most of the time, and data seems to be accurate in as much as I double checked:
'Actions loop test' for the 'Testing' package was found in 'modules/simpletest/ ' but a previous module was in 'modules/update/ '.
'Aggregator module tests' for the 'Testing' package was found in 'modules/aggregator/ ' but a previous module was in 'modules/update/ '.
'AJAX form test mock module' for the 'Testing' package was found in 'modules/simpletest/ ' but a previous module was in 'modules/update/ '.
'AJAX Test' for the 'Testing' package was found in 'modules/simpletest/ ' but a previous module was in 'modules/update/ '.
'Batch API test' for the 'Testing' package was found in 'modules/simpletest/ ' but a previous module was in 'modules/update/ '.
'Block test' for the 'Testing' package was found in 'modules/block/ ' but a previous module was in 'modules/update/ '.

Thanks for the patch Jason!

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Title: Sitedoc: D7 port » Port SiteDoc to Drupal 7
Issue summary: View changes
Status: Postponed » Active

So it sounds like the patch just needs a little sprucing up?