Home » Projects » Ubercart Contributed Modules » Ubercart Contributions » Issues

attributes

Submitted by agilpwc on Tue, 01/29/2008 - 16:22
Project:Ubercart Contributions
Component:Code
Category:
Priority:normal
Assigned:Unassigned
Status:active
Description
Project: 
Ubercart

Latest bazaar I just download a few minutes ago.

Was on the attribute options page for a product class, choosing defaults and stuff.
When I save I got this error
user warning: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'AND oid = 0' at line 1 query: DELETE FROM uc_class_attribute_options WHERE pcid = AND oid = 0 in www/includes/database.mysql.inc on line 172.

It saved my choices, not sure what didn't get deleted though?

By the way is it better to post bug reports in the forum or here at issue tracker?
Tue, 01/29/2008 - 16:32
#1 View page
Lyle
Lyle's picture
Offline
AdministratoreLiTe!
Joined: 08/07/2007
Juice: 6846
Re: attributes

I knew I'd miss one of those. Too bad beta 4 is already out. Sad

I like the issue tracker better for bug reports, especially this kind that don't need much discussion. We'd also like to use the drupal.org issue tracker, but I don't know how that's going to pan out.

Top
Thu, 01/31/2008 - 01:41
#2 View page
justin
justin's picture
Offline
Joined: 01/31/2008
Juice: 22
A fix
Assigned to:Lyle» justin

I've submitted a fix for this. See:
http://www.ubercart.org/issue/2893/bug_fix_attributes

Top
Thu, 01/31/2008 - 10:13
#3 View page
Lyle
Lyle's picture
Offline
AdministratoreLiTe!
Joined: 08/07/2007
Juice: 6846
Re: A fix
Assigned to:justin» Lyle

Actually, I'd already fixed this, I just forgot to mark the issue. Thanks for the effort, but why did you make a whole new issue for it? I would have expected you to add a comment to this one.

Top
Thu, 01/31/2008 - 18:51
#4 View page
justin
justin's picture
Offline
Joined: 01/31/2008
Juice: 22
Re: Re: A fix
Assigned to:Lyle» justin

Sorry, not very familiar with the due process of these things. Sadly most of my work is done for a company and not for the open source community. Also, I had planned on submitting some other patches on that post but after looking into it further, I realized the 'bugs' I discovered were actually by design.
I suppose in that case it would have been better to submit separate posts for each issue though eh? Anyway, glad it's fixed.

Top