Title says it all,patch attached

CommentFileSizeAuthor
#3 use-vid-instead-of-nid-1366378.patch789 bytesSeanA
values_wont_show_in_node_edit.patch794 bytesParisLiakos
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SeanA’s picture

SeanA CreditAttribution: SeanA commented

This doesn't seem to be an issue with the latest dev release. Role price fields show up on the node edit form as expected. I can't reproduce this.

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented

hmm,actually the fields where showing up just fine..the values are the problem.assign a value to a role,save the node..are the values displaying when editing the node?
i had this problem with the same release since the last commit was in september

SeanA’s picture

SeanA CreditAttribution: SeanA commented
FileSize
use-vid-instead-of-nid-1366378.patch789 bytes

OK, yes, this is a bug. Once I added a new revision an error appeared: "Notice: Undefined property: stdClass::$sell_price in uc_price_per_role_node_load() (line 297" and the role price was indeed missing. The patch by rootatwc fixes the problem.

But... do we still need to retrieve nid from the database? It's not being used so using just vid works.

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented
Status: Needs review » Reviewed & tested by the community

yes..much better

SeanA’s picture

SeanA CreditAttribution: SeanA commented

Looks like a related issue here: #1420394: Role Price Not Showing Up In Cart or Checkout

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

This patch worked for me, thanks.

DrupalDummie’s picture

DrupalDummie CreditAttribution: DrupalDummie commented

That fixed it!!! The patch in #3

Dan Z’s picture

Dan Z CreditAttribution: Dan Z commented

#1831360: Notice: Undefined property: stdClass::$sell_price when creating new product version closed as duplicate.

DanZ’s picture

DanZ CreditAttribution: DanZ commented
Status: Reviewed & tested by the community » Fixed

Committed #3. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.