Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Even if subcategories have nodes, main category count is still zero.
PS : Main category's page (therefore unreachable) shows correctly subcategories nodes.
Comment | File | Size | Author |
---|---|---|---|
#8 | categorycount-1593570-8.patch | 2.62 KB | daffodilsoftware |
#6 | categorycount-1593570-6.patch | 2.62 KB | daffodilsoftware |
#4 | categorycount-1593570-4.patch | 2.67 KB | daffodilsoftware |
#1 | site_map-count_children-1593570-1.patch | 2.43 KB | Avorathol |
site-map-count-0.png | 3.32 KB | Avorathol |
Comments
Comment #1
Avorathol CreditAttribution: Avorathol commentedPatchfile for count children against 7.x-1.x branch. It adds a configuration option and count children if specified.
Comment #2
darrell_ulm CreditAttribution: darrell_ulm commentedThanks very much for the patch Avorathol. Hope to test and apply soon.
Comment #3
darrell_ulm CreditAttribution: darrell_ulm commentedAvorathol,
The patch does not apply, can you check and/or fix and re-post?
Thank you,
Comment #4
daffodilsoftware CreditAttribution: daffodilsoftware commentedAvorathol,
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.
Comment #5
frjo CreditAttribution: frjo commentedPlease rework the patch so it follows the Drupal coding standard.
http://drupal.org/coding-standards
Use an indent of 2 spaces, with no tabs for example.
Comment #6
daffodilsoftware CreditAttribution: daffodilsoftware commentedSorry It was my mistake there was only one problem of a trailing whitespace.adding the patch again
Comment #7
frjo CreditAttribution: frjo commentedI'm afraid there are a number of other coding standard problems with the patch. Mostly these http://drupal.org/coding-standards#controlstruct.
Comment #8
daffodilsoftware CreditAttribution: daffodilsoftware commentedI revised the patch as recommended.And here it goes.couple of braces were misplaced right?
Comment #9
darrell_ulm CreditAttribution: darrell_ulm commentedThis 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.
Comment #10
darrell_ulm CreditAttribution: darrell_ulm commented