Skip to:
Content

BuddyPress.org

Opened 17 years ago

Closed 17 years ago

#1026 closed defect (bug) (fixed)

Inconsistence between param name in xprofile_insert_field() and BP_XProfile_Field attribute

Reported by: luka.bernardi Owned by:
Priority: minor Milestone: 1.1
Component: Version:
Severity: Keywords:
Cc:

Description

xprofile_insert_field() expect an assoc array, one of this key is field_group_id and is mapped in BP_XProfile_Field with

$field->field_group_id = $field_group_id;

but field_group_id isn't an BP_XProfile_Field's attribute nor a column in wp-bp-xprofile-field table.
Instead of field_group_id use group_id

Change History (5)

#1 @djpaul
17 years ago

  • Milestone1.1
  • Priority majorminor

#2 @apeatling
17 years ago

  • Resolutionfixed
  • Status newclosed

(In [1898]) Fixes #1026 props luka.bernardi

#3 @luka.bernardi
17 years ago

  • Resolution fixed
  • Status closedreopened

Can I suggest to accept group_id as a key of the array,otherwise the naming conventions could become inconsistent?

#5 @apeatling
17 years ago

  • Resolutionfixed
  • Status reopenedclosed

Actually, I'm not sure exactly what you mean looking at it now. The member var in the field class is group_id, and so is the field in the DB. I don't really want to change that. Everyone will be interfacing with the class through the xprofile_insert_field() function, so it shouldn't matter a great deal.

Note: See TracTickets for help on using tickets.