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.Problem/Motivation
The docblock for template_preprocess_poll_bar() has a @see theme_poll_bar() in it. That function doesn't exist.
Proposed resolution
Remove '@see theme_poll_bar()' from https://api.drupal.org/api/drupal/modules%21poll%21poll.module/function/...
Remaining tasks
Remove '@see theme_poll_bar()'.
User interface changes
None.
API changes
None.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | 2324821-13.patch | 412 bytes | rpayanm |
| #13 | 2324821-11.patch | 412 bytes | rpayanm |
| #2 | drupal7-remove-theme-poll-bar-2324821-2.patch | 412 bytes | er.pushpinderrana |
Comments
Comment #1
Elijah LynnComment #2
er.pushpinderrana CreditAttribution: er.pushpinderrana commentedPlease review.
Comment #3
Elijah LynnThanks!
Comment #4
dcam CreditAttribution: dcam commentedLooks good. RTBC +1.
Comment #5
Elijah LynnComment #6
Elijah LynnComment #9
dcam CreditAttribution: dcam commentedComment #12
dcam CreditAttribution: dcam commentedComment #13
rpayanmTrying...
Comment #14
rpayanmSorry for the two patches, are identical :)
Comment #15
dcam CreditAttribution: dcam commented@rpayanm
I have the impression that you uploaded these patches because you saw #2 fail earlier. That failure was due to a Testbot crash or some other kind of system failure. Unfortunately, those kinds of failures have been plaguing us for months. Testbot re-tests RTBC patches routinely, but then some of the time it crashes like that. When it has one of those failures for a 7.x patch, then it just has to be tested yet again and set back to RTBC. There's no need to reroll the patch or re-upload it. #2 is still good.
Comment #16
rpayanm@dcam That's right, now I understanding :)
Grettings.
Comment #17
David_Rothstein CreditAttribution: David_Rothstein commentedCommitted to 7.x - thanks!