Even if subcategories have nodes, main category count is still zero.

PS : Main category's page (therefore unreachable) shows correctly subcategories nodes.


Avorathol’s picture

Priority: Major » Normal
Status: Active » Needs review
2.43 KB

Patchfile for count children against 7.x-1.x branch. It adds a configuration option and count children if specified.

darrell_ulm’s picture

Thanks very much for the patch Avorathol. Hope to test and apply soon.

darrell_ulm’s picture

Status: Needs review » Needs work


The patch does not apply, can you check and/or fix and re-post?

File to patch: site_map.module
patching file site_map.module
Hunk #1 succeeded at 435 with fuzz 1 (offset -75 lines).
Hunk #2 FAILED at 604.

Thank you,

daffodilsoftware’s picture

I also tried the patch that you created but it failed.I reworked on your code to get it functional and thus created a new patch so Avorathol/darrellulm Please test it and give your reviews.

frjo’s picture

Assigned: Unassigned » daffodilsoftware

Please rework the patch so it follows the Drupal coding standard.


Use an indent of 2 spaces, with no tabs for example.

daffodilsoftware’s picture

Sorry It was my mistake there was only one problem of a trailing whitespace.adding the patch again

frjo’s picture

I'm afraid there are a number of other coding standard problems with the patch. Mostly these http://drupal.org/coding-standards#controlstruct.

daffodilsoftware’s picture

I revised the patch as recommended.And here it goes.couple of braces were misplaced right?

darrell_ulm’s picture

This is the largest patch, I should have tested this one 1st, but it didn't work out that way. Committed another one of your patches which makes this one not apply. Are you willing to do a quick re-roll of you have all the sources avail? Thanks!

If this one passes, we ought to apply it 1st as it has the most changes.

darrell_ulm’s picture

Issue summary: View changes
Status: Needs work » Needs review