Move product field ordering to standard field display management

I doubt we'll put a change this significant into 2.0 now that we've moved into RC territory. However, the goal for 3.0 is definitely to introduce a dependency on CCK, so that problem should solve itself.

We can always review what you come up with, but we're pretty aggressively pushing for a 2.0 release, so the patches we're committing are more about removing and fixing things than adding in new functionality.

Moving to the 7.x-3.x queue. I believe this is being addressed now, as it's a natural consequence of moving from CCK to Fields in Drupal 7.

Sub

subscribe

actually just realizing this is more important to me.
is there a patch anywhere that allows us to order all the fields on a product node?
I have fields showing up in awkward places and I cannot move them.

Thanks for any help,

Jeff

Is this patch the solution for Ubercart 2.2?

http://drupal.org/node/333085

apparently, not. just installed it a second time and could not get it to work.

Here's a good module related to this thread's discussion (adding custom CCK field attributes to Ubercart products)...
http://drupal.org/node/923708

Here's a good module related to this thread's discussion (adding custom CCK field attributes to Ubercart products)...
http://drupal.org/node/923708

I put it as a module here: http://drupal.org/project/uc_attribute_cck

Patch attached that removes the separate product field settings page and uses the standard field settings page instead. Also fixes up the teaser display settings when uc_product is installed or upgraded. Needs testing!

i just installed ubercart 2.4 and was surprised that this was not fixed in that release either.

A patch for 2.x can be found in #333085: Field weight order using CCK interface but is not yet ready to commit.

Updated patch against latest -dev, and changing this to a bug after #1184110: SKU does not display

Adding tag; we should fix this before release.

To apply this patch I must use 7.x-3.x-dev ?

I can't applied the patch against 7.x-3.x-dev:

$ patch -p1 < ../443926-product-field-display_0.patch
patching file uc_product/uc_product.admin.inc
patching file uc_product/uc_product.install
patching file uc_product/uc_product.module
patching file uc_product_kit/uc_product_kit.install
patching file uc_product_kit/uc_product_kit.module
Hunk #3 FAILED at 643.
1 out of 3 hunks FAILED -- saving rejects to file uc_product_kit/uc_product_kit.module.rej

Thanks!

You need the -dev release from git, the tarball hasn't yet updated with a change I committed earlier. But you still should be able to test products with the above failure, product kits won't work quite right though.

I have applied the patch but the bug #1184110: SKU does not display still persist

For me, once the patch is installed, I can visit /admin/structure/types/manage/product/display/teaser and change SKU to Visible to make it show up on teasers. So you're saying this doesn't happen for your case?

Well, If I change (in /admin/structure/types/manage/product/display/teaser) SKU to Visible it shows "only" the label of SKU but not its value.
I have the same problem in the full node.

Reopening your other issue, as this patch is for a different problem; at least the SKU label shows up on teasers with this patch in place, which is a little better than before.

After I have update the module with this patch is disappeared the content of Catalog block.

Strange, that doesn't happen for me, and this patch doesn't touch that code. Does the block not show up at all, or is the title there but the rest is empty?

The rest is empty.

I restored the beta and the block works fine.

And it goes back to normal if you revert the patch?

It's strange, works only with "my" previous Ubercart 3.0-beta3, 2011-5-26 ...if I download again Ubercart 3.0-beta3, 2011-5-26 it not works but if I use my backup works.

Committed this, as having two separate places to manage product display fields, one of which didn't work as expected, and one of which was inflexible compared to Drupal core, just didn't make sense.

I can't reproduce the above catalog issue, please open a new issue if this is still a problem.