New lower pricing for Subscriptions

50%off old price

move new person to existing household deletes persons record

About Support Directory move new person to existing household deletes persons record

  • This topic is empty.
Viewing 4 posts - 1 through 4 (of 4 total)
  • Author
    Posts
  • #2524
    Avatar for mkirchhoefermkirchhoefer
    Member

    Hi,

    I’mm running XAMPP 1.8.3 (phpinfo attached)
    each time I try to move a freshly created person to an existing houshold, that person’s record is deleted form wp_church_admin_people table and there is an empty household record in wp_church_admin_household.

    If I try to add a person directly to a household everything works as expected.

    I already tried to do some debugging in directory.php:
    $wpdb->query(‘DELETE FROM ‘.CA_PEO_TBL.’ WHERE
    household_id=”‘.esc_sql($data->household_id).'”‘);

    the query-string looks like:
    DELETE FROM wp_church_admin_people WHERE
    household_id=”18″

    and
    $wpdb->query(‘UPDATE ‘.CA_PEO_TBL.’ SET
    household_id=”‘.esc_sql($_POST).'” WHERE
    people_id=”‘.esc_sql($id).'”‘);

    the query-string looks like
    query(UPDATE wp_church_admin_people SET household_id=”7″ WHERE
    people_id=”7″)

    From my understanding of SQL that does not look too bad

    in wp-config.php WP_LANG is set: define(‘WPLANG’, ‘de_DE’);

    church_admin/index.php contains:
    setlocale(LC_TIME, WPLANG);

    any idea?

    Thanks

    Martin

    #3237
    Avatar for Andy MoyleAndy Moyle
    Keymaster

    Thanks for that – I can immediately see what is wrong!

    If there is only one person in the household, instead of deleting the old household it is deleting the people record. Fixed for next update!

    #3238
    Avatar for mkirchhoefermkirchhoefer
    Member

    okay, see it: looks as if the wrong table is modified CA_PEO_TBL instead of CA_HOU_TBL

    Thanks

    Martin

    #3239
    Avatar for Andy MoyleAndy Moyle
    Keymaster

    That’s correct – it’s in the next update!

Viewing 4 posts - 1 through 4 (of 4 total)
  • You must be logged in to reply to this topic.