Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I have a view table with group.
In some cases the serial increment (field: Global view result counter) is not consistent when the setting of the table has grouping of the field.
Attached the table showing the problem in Global view result counter.
Comment | File | Size | Author |
---|---|---|---|
#3 | screenshot-localhost 2016-03-15 16-54-08.png | 49.59 KB | eradeepak |
#2 | theme.inc_.patch | 761 bytes | eradeepak |
table-counter-problem.PNG | 6.26 KB | waluyo.umam |
Comments
Comment #2
eradeepak CreditAttribution: eradeepak as a volunteer commentedHi Waluyo,
I have created patch file with auto increment in drupal views table for group by field.
Please review it. Hope this will help you.
Comment #3
eradeepak CreditAttribution: eradeepak as a volunteer commentedComment #4
waluyo.umam CreditAttribution: waluyo.umam as a volunteer commentedThank you.
Yes, It works. Now the numbering is correct in each group.
Comment #5
eradeepak CreditAttribution: eradeepak as a volunteer commentedComment #7
eradeepak CreditAttribution: eradeepak as a volunteer commentedComment #8
waluyo.umam CreditAttribution: waluyo.umam as a volunteer commentedWhat's the problem with this patch?
I modified the codes following this patch and it works. Until today not included in the release.
Comment #9
eradeepak CreditAttribution: eradeepak as a volunteer commentedComment #10
zalak.addweb CreditAttribution: zalak.addweb commentedComment #11
waluyo.umam CreditAttribution: waluyo.umam as a volunteer commentedEvery time updated views, then I have to manually modify the codes to maintain the numbering correct. I hope this patch can be implemented.
Comment #13
TR CreditAttribution: TR commented@was.uthm: The patch failed to apply 2 years ago and still fails to apply. If you would like to see this committed, please re-roll the patch and post it here so the testbot can review it. Then fix any coding standards or other issues identified by the testbot. Code comments and a test case would be nice too ... Once we have a working patch that can apply and pass the tests, then it can be reviewed and potentially committed.
Note that patches must be applied to the most recent development version, so I moved the version to 7.x-3.x.
Back to "Needs work". I also removed the tags. Please see https://www.drupal.org/node/1023102 for guidelines on the proper use of tags. Specifically, you don't need a "views" tag - this is the Views queue. You don't need a "counter" tag, that is just a keyword which is already used in the issue. And "ChennaiDrupalGroup" may have been relevant at one point, but that group hasn't touched this patch in 2 years so it is unlikely they are still working on it - they can add that tag back if they want to resume their participation in this issue.
Comment #14
marksmith CreditAttribution: marksmith commentedPatch #2 is breaks the reverse views results counter function.