Skip to:

Opened 15 years ago

Closed 13 years ago

#1024 closed enhancement (fixed)

xprofile_insert_field() should return the field_id

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


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 13 years ago.

Download all attachments as: .zip

Change History (7)

#1 @djpaul
15 years ago

  • Milestone set to 1.2

#2 @DJPaul
14 years ago

  • Milestone changed from 1.2 to 1.3

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

#3 @johnjamesjacoby
14 years ago

  • Component set to Core
  • Owner set to johnjamesjacoby
  • Status changed from new to assigned

Assigning to myself for xprofile 1.3 make-over.

#4 @DJPaul
13 years ago

  • Component changed from Core to XProfile
  • Keywords dev-feedback added

John, did this get looked at?

#5 @cnorris23
13 years ago

  • Keywords has-patch added; dev-feedback removed

It hasn't yet been fixed. Patch attached.

13 years ago

#6 @djpaul
13 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

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

Note: See TracTickets for help on using tickets.