Closed (fixed)
Project:
UC Profile
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Jul 2010 at 09:30 UTC
Updated:
26 Jul 2011 at 04:51 UTC
Jump to comment: Most recent file
Comments
Comment #1
derhasi commentedThe problem is that the temporary order values are not assigned to the
$profile_form. So I added a mapping function to put any temporary order value in the profile form's default values.The patch is attached and needs review.
Comment #2
EvanDonovan commentedPatch looks logical; didn't get a chance to test yet. If #606366: Make it possible for uc_profile to include fields from different profile categories lands, it will need to be re-rolled to match the new array structure.
Comment #3
EvanDonovan commentedJust realized I need this. I will re-roll into #606366: Make it possible for uc_profile to include fields from different profile categories on Monday, probably.
Comment #4
EvanDonovan commentedIn #606366-25: Make it possible for uc_profile to include fields from different profile categories, I have a version that works with multi-category support. Hopefully, that can become a 2.x branch of the module soon.
Comment #5
pcorbett commented@EvanDonovan, is this all set for the latest dev branch? If so, we can mark as fixed.
Comment #6
EvanDonovan commentedYes, I put this in my 2.x, so this is fixed in there. I think 2.x should become the recommended branch soon; it was tested by a lot of people prior to commit.