Skip to:
Content

BuddyPress.org

Opened 17 years ago

Closed 16 years ago

#1024 closed enhancement (fixed)

xprofile_insert_field() should return the field_id

Reported by: luka.bernardi Owned by: johnjamesjacoby
Priority: minor Milestone: 1.5
Component: Extended Profile Version:
Severity: Keywords: has-patch
Cc:

Description

For coerence if the xprofile_get_field() and xprofile_delete_field() accept only the field_id, the function xprofile_insert_field() should return, in case of success, the field's id just created.

Attachments (1)

1024.001.diff​ (491 bytes ) - added by cnorris23 16 years ago.

Download all attachments as: .zip

Change History (7)

#1 @djpaul
17 years ago

  • Milestone → 1.2

#2 @DJPaul
17 years ago

  • Milestone 1.2 → 1.3

Still valid, but bumping to 1.3 as this might break too much stuff for 1.2.1

#3 @johnjamesjacoby
16 years ago

  • Component → Core
  • Owner set to johnjamesjacoby
  • Status new → assigned

Assigning to myself for xprofile 1.3 make-over.

#4 @DJPaul
16 years ago

  • Component Core → XProfile
  • Keywords dev-feedback added

John, did this get looked at?

#5 @cnorris23
16 years ago

  • Keywords has-patch added; dev-feedback removed

It hasn't yet been fixed. Patch attached.

@cnorris23
16 years ago

#6 @djpaul
16 years ago

  • Resolution → fixed
  • Status assigned → closed

(In [4121]) Make xprofile_insert_field return the field_id. Fixes #1024, props cnorris23

Note: See TracTickets for help on using tickets.