[buddypress-trac] [BuddyPress Trac] #5919: Xprofile field meta values not deleted with field

buddypress-trac noreply at wordpress.org
Fri Oct 3 14:18:18 UTC 2014

#5919: Xprofile field meta values not deleted with field
 Reporter:  tometzky                    |       Owner:
     Type:  defect (bug)                |      Status:  new
 Priority:  low                         |   Milestone:  Future Release
Component:  XProfile                    |     Version:  2.1
 Severity:  minor                       |  Resolution:
 Keywords:  needs-patch good-first-bug  |

Comment (by tometzky):

 Yeah - it should be easy... But.


 When I added in `BP_XProfile_Field::delete()`:

 if ( !bp_xprofile_delete_meta( $this->id, 'field' ) ) return false;

 Then it breaks when there's no meta defined.


 Then I tried to check if any metadata exists. And then I went on to read
 bp_xprofile_filter_meta_query() and bp_filter_metaid_column_name()
 functions. And I cried. They're blindly replacing substrings in SQL query.
 This is [http://thedailywtf.com/ thedailywtf.com] inspired coding.

Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5919#comment:2>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac

More information about the buddypress-trac mailing list