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.
With i18ntaxonomy enabled, you get a PHP error in the latest release when using hs_taxonomy - it's a really minor thing, but pretty critical:
// Line 282 && 315
$vid = $vocabulary->$vid;
should become:
$vid = $vocabulary->vid;
Comment | File | Size | Author |
---|---|---|---|
#1 | empty_property.patch | 927 bytes | swentel |
Comments
Comment #1
swentel CreditAttribution: swentel commentedWith patch ..
Comment #2
Wim LeersUGH. Boy, do I hate the i18n module. Still a major source of Drupal suckage. If it were built better, I wouldn't have to be frustrated by duplicating code that shouldn't exist in the first place.
Fixed.
http://drupal.org/cvs?commit=389714
Comment #3
Kevin P Davison CreditAttribution: Kevin P Davison commentedHi Wim,
I'm still getting "Fatal error: Cannot access empty property in /sites/default/modules/hierarchical_select/modules/hs_taxonomy.module on line 282" in 3.3 and had to go back to 3.1 so we can edit nodes.
* using hs_taxonomy+i18n (6.x-1.3)
* Pressflow 6.16
I can provide more detail, and you're probably going to ask me to update i18n first?
Thank you,
Kevin
Comment #4
Wim Leers@Quevin_LLC: I pinged swentel to reply.
Comment #5
swentel CreditAttribution: swentel commentedHaven't downloaded the 3.3 version, applied the patch myself and didn't have that problem any more. @Quevin_LLC, can you look at line 282 and 315 and see what it looks like ? If you still see
than your upgrade somehow failed. I'll do the upgrade on monday to see what happens, but I do think the upgrade simply failed.
Comment #6
Wim LeersClosing due to lack of response.