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.
In a new drupal install, i installed UTL and created league, division and teams.
I created some games, submitted the results and i ran cron. The result was this error:
Table 'league.content_field_ult_leaguestatus' doesn't exist query: Select a.nid from content_type_ult_division a, content_field_ult_leaguestatus b where a.nid = b.nid and b.field_ult_leaguestatus_value = "Active" in /home/theodim/web/league/sites/all/modules/ULT/modules/ult_standings/ult_standings.module on line 358.
And in the standings page, i get this
No Standings Available
.
Comment | File | Size | Author |
---|---|---|---|
#9 | ult_standings.fix-registration-table-dependency-1214358-9.patch | 992 bytes | jthorson |
#2 | ult_standings-fix-registrations-table-dependency-1214358-2.patch | 1020 bytes | jthorson |
Comments
Comment #1
jthorson CreditAttribution: jthorson commentedThanks for the report!
My first guess is that the combination of modules you've enabled has flushed out a built-in dependency I didn't consider ... Can you confirm whether or not you've enabled the 'ult_registration' module?
Comment #2
jthorson CreditAttribution: jthorson commentedPatch attached ... fix commited to 6.x-1.x-dev.
Comment #3
Liakoni CreditAttribution: Liakoni commentedYes, i don't have ULT_Registration module enable. I'll try with the patch.
Comment #4
jthorson CreditAttribution: jthorson commentedSounds good ... let me know if it works for you!
Comment #5
mstrelan CreditAttribution: mstrelan commentedPatch definitely doesn't work. The $table variable is not defined.
Comment #6
jthorson CreditAttribution: jthorson commentedThat's odd ... The $table variable is defined in lines 346 through 351 of the ult_standings module:
This function is only called from two places ... lines 153 and 157 of ult_standings.module. One instance calls it with no arguments (which causes the default $type="league" argument to be assumed), and the other calls it with the argument "division". Therefore, one of the first two 'if' conditions should always be invoked, and thus the $table variable should always be defined.
I wonder if perhaps the patch didn't apply properly (or was applied against an earlier version of the code) ... Can you verify that your resulting function looks like the above, after the patch has been applied?
Comment #7
mstrelan CreditAttribution: mstrelan commentedI think you're looking at the wrong function. The patch makes changes to ult_standings_getcurrentsession() however you're referring to ult_standings_getactiveleagues()
Comment #8
jthorson CreditAttribution: jthorson commentedYup ... you're right ... the patch is applying against the wrong function. Not sure how that happened ... I'll re-roll and get something back here in about 20 minutes.
Comment #9
jthorson CreditAttribution: jthorson commentedUpdated patch ... let me know if it works, and sorry for the trouble!
Comment #10
mstrelan CreditAttribution: mstrelan commentedWorks well, thanks
Comment #11
jthorson CreditAttribution: jthorson commentedReverted bad patch and applied new one ... fix commited to 6.x-1.x.
Thanks for the heads up!
Comment #13
jthorson CreditAttribution: jthorson commentedThis fix was incomplete ... also see the patch posted at http://drupal.org/node/1529370#comment-5884418