Closed (duplicate)
Project:
Ubercart
Version:
6.x-2.0-rc3
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Apr 2009 at 21:42 UTC
Updated:
26 Oct 2009 at 11:03 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
pirofun commentedalso getting this error.
commented out line 1419 and that seems to restore functionality, however, there needs to be a correct fix applied.
Comment #2
pirofun commentedworked for me:
http://drupal.org/node/439234
Comment #3
Island Usurper commentedReopen, please, if this isn't the same issue.
Comment #4
mkalkbrennerWe have the same problem but don't use product kit.
So here's a quick fix. But it seems to me that the module design should be reviewed ...
Comment #5
Island Usurper commentedThat issue is less about the product kit module and more about Meta tags rendering the node bodies before the bootstrap is complete. Both uc_product and uc_product_kit are doing the same thing, so having Meta tags set to using the teaser when the description is empty will break both of them.
Comment #6
swill commentedI can confirm this bug still exists in RC3...
For me it only happens on products that have restricted access using ACL.
Comment #7
swill commentedi can also confirm that adding this to the top of the function theme_uc_product_price solved my problem...
Comment #8
Island Usurper commentedYes, that would work, but it should be totally unnecessary. That exact code is being run in uc_store_init(), and I would expect that nodes shouldn't be viewed until after that happens.
I can't duplicate the error with ACL. How exactly are you restricting access to those products? Like I said in the other thread, I don't see how they can be the problem, but it doesn't hurt to be thorough.
Comment #9
gosvald commentedI'm sorry swill but which file/function did you add this code to?
Comment #10
matt v. commentedI was getting the following error running Ubercart 6.x-2.0-rc3 and Meta Tags 6.x-1.0:
Applying the patch from Comment #4 above fixed the issue. For what it's worth, we aren't doing anything special in terms of restricting access to products.
Comment #11
Anonymous (not verified) commentedI have the same problem.
For the first product I've created for my first UC store !
Bad luck !
I can't show this product : this is a big problem !
Here is the current configuration (thanks to Configuration Documentation module)
Comment #12
Island Usurper commentedYou also have the Meta Tags module enabled, and the default configuration causes a known issue as described in the link in #2 above.
No one's allowed to mention this problem if they are using Meta Tags, at least until my patch at #439234-22: Replace nodewords_init() with nodewords_preprocess_page() (or later) gets committed.
Comment #13
bramface commentedI'm noticing that the includes file referenced in this patch, ./includes/uc_price.inc , is not actually (on my system) in that referenced location. Relative to /ubercart/uc_product/uc_product.module, it is in ../uc_store/includes/uc_price.inc .
So - I've rewritten that reference and it seems to solve the first problem - but I'm wondering if maybe somehow uc_price.inc ended up in the wrong place some earlier time...now I need to chase down the reference in uc_product_kit.....
-Bram
Comment #14
tr commentedbmoreinis: What are you talking about? Do you mean the code in #7 (which is not a patch?). If so, that code is written correctly. However I would recommend Island Ursurper's fix at http://drupal.org/node/439234#comment-1588868 as the proper way to go about dealing with this problem.
Comment #15
dave_______1 commentedHi everyone,
The code above posted by swill seems to work for me.
Here is what I did:
Under it type the lines
$dir = drupal_get_path('module', 'uc_store');
require_once($dir .'/includes/uc_price.inc');
(take out the
this is only so you can read the code)Hope this helps someone.
Dave.
Comment #16
Island Usurper commentedThis is redundant and unhelpful. A fix for the Meta Tags module has been made in the 6.x-1.x-dev version, so any hacking of Ubercart is not needed to deal with this error.
Comment #17
KAP10 commentedThank you so much mkalkbrenner. I used the fix quoted in comment 4 and the problem just vanished. After I inserted the code that mkalkbrenner quoted in to uc_product.module, our site, VolidayGreetings.com is working just fine again.
Comment #18
kenorb commentedSolution:
Download the latest version of nodewords module:
http://drupal.org/node/222124
Comment #19
joachim commentedExcept the 6-2 branch is, as written in the release notes, unstable.
The fix has been applied by the maintainer to the 6-1 branch too but there's no point release or dev release of that yet.
Comment #20
John Tolhurst commentedI need metatags, or my google ranking slides.
http://drupal.org/node/222124 cannot be used on a production site.
So am I right therefore to conclude that I must indeed patch uc_product.module according to the procedure in post #15? by dave_______1 ?
Thanks for the assistance here.
Comment #21
joachim commentedSolution: get the maintainer of NW to support his users by making a 6-1 release.
Comment #22
Fohsap commentedWhat are we supposed to do with this? Where does it go?
Comment #23
Fohsap commentedSwill... where do you add that?
Comment #24
Fohsap commentedDave's fix doesn't work.
Comment #25
skydesigner commentedDave's fix is work !!!! Thanks Dave...
Comment #26
dave_______1 commentedHi Everyone,
This fix is from swill, not myself, so all credit to him (or her).
I've just had the same problem again after upgrading some of the modules on a new site I'm building, but after aplying the fix by swill, it's all fine.
The line number has shifted though (to 1523), so please search for the function theme_uc_product_price to apply.
Great modules, great fixes, great community, Drupal is just great (try osCommerce or Joomla then you'll agree too).
Thanks everyone for making Drupal so good!
Dave.
Comment #27
alexmoreno commentedi solved the problem simply upgrading nodewords (seems the cause)