Problema update mysql in php

[ShanReWeT]

Salve a tutti,
sto creando un piccolo script in php per SMF.
Lo script funziona solo che da qualche problema

updateMemberData($row['ID_MEMBER'], array('avatar' => 9, 'validation_code' => '\'\''));

così funziona notate l'array avatar = 9
mentre se metto

updateMemberData($row['ID_MEMBER'], array('avatar' => 'foto.jpg', 'validation_code' => '\'\''));

avatar = 'foto.jpg' sul database appare bianco.
quale può essere la causa?

[bstefano79]

il primo è un numero (9) il secondo una strina ('foto.jpg'), controlla di che tipo è il campo sul db, vedrai che è numrico, cambialo in varchar

[serdominik]

poi se per caso il campo del database pensi che vai a salvare stringe molto lunghe ti conviene di impostare il campo in TEXT
perchè in varchar il massimo è 255

[ShanReWeT]

Originariamente inviato da serdominik
poi se per caso il campo del database pensi che vai a salvare stringe molto lunghe ti conviene di impostare il campo in TEXT
perchè in varchar il massimo è 255

non è un problema di database perchè dal sito tranquillamente carico avatar e nel database viene inserito url. Il problema è in quella stringa php.

[serdominik]

xkè non posti la funzione: updateMemberData
nella sua interezza? potrebbe esserci qualcosa di errato all'interno

[ShanReWeT]

Originariamente inviato da serdominik
xkè non posti la funzione: updateMemberData
nella sua interezza? potrebbe esserci qualcosa di errato all'interno

eccolo

function updateMemberData($members, $data)
{
    global $db_prefix, $modSettings, $ID_MEMBER, $user_info;

    if (is_array($members))
        $condition = 'ID_MEMBER IN (' . implode(', ', $members) . ')
        LIMIT ' . count($members);
    elseif ($members === null)
        $condition = '1';
    else
        $condition = 'ID_MEMBER = ' . $members . '
        LIMIT 1';

    if (isset($modSettings['integrate_change_member_data']) && function_exists($modSettings['integrate_change_member_data']))
    {
        // Only a few member variables are really interesting for integration.
        $integration_vars = array(
            'memberName',
            'realName',
            'emailAddress',
            'ID_GROUP',
            'gender',
            'birthdate',
            'websiteTitle',
            'websiteUrl',
            'location',
            'hideEmail',
            'timeFormat',
            'timeOffset',
            'avatar',
            'lngfile',
        );
        $vars_to_integrate = array_intersect($integration_vars, array_keys($data));

        // Only proceed if there are any variables left to call the integration function.
        if (count($vars_to_integrate) != 0)
        {
            // Fetch a list of memberNames if necessary
            if ((!is_array($members) && $members === $ID_MEMBER) || (is_array($members) && count($members) == 1 && in_array($ID_MEMBER, $members)))
                $memberNames = array($user_info['username']);
            else
            {
                $memberNames = array();
                $request = db_query("
                    SELECT memberName
                    FROM {$db_prefix}members
                    WHERE $condition", __FILE__, __LINE__);
                while ($row = mysql_fetch_assoc($request))
                    $memberNames[] = $row['memberName'];
                mysql_free_result($request);
            }

            if (!empty($memberNames))
                foreach ($vars_to_integrate as $var)
                    call_user_func($modSettings['integrate_change_member_data'], $memberNames, $var, stripslashes($data[$var]));
        }
    }

    foreach ($data as $var => $val)
    {
        if ($val === '+')
            $data[$var] = $var . ' + 1';
        elseif ($val === '-')
            $data[$var] = $var . ' - 1';
    }

    // Ensure posts, instantMessages, and unreadMessages never go below 0.
    foreach(array('posts', 'instantMessages', 'unreadMessages') as $type)
        if (isset($data[$type]) && preg_match('~^' . $type . ' - ([\d]+)~', $data[$type], $match) === 1)
            $data[$type] = 'CASE WHEN ' . $type . ' <= ' . $match[1] . ' THEN 0 ELSE ' . $data[$type] . ' END';

    $setString = '';
    foreach ($data as $var => $val)
    {
        $setString .= "
            $var = $val,";
    }

    db_query("
        UPDATE {$db_prefix}members
        SET" . substr($setString, 0, -1) . '
        WHERE ' . $condition, __FILE__, __LINE__);

    updateStats('postgroups', $condition, array_keys($data));

    // Clear any caching?
    if (!empty($modSettings['cache_enable']) && $modSettings['cache_enable'] >= 2 && !empty($members))
    {
        if (!is_array($members))
            $members = array($members);

        foreach ($members as $member)
        {
            if ($modSettings['cache_enable'] == 3)
            {
                cache_put_data('member_data-profile-' . $member, null, 120);
                cache_put_data('member_data-normal-' . $member, null, 120);
                cache_put_data('member_data-minimal-' . $member, null, 120);
            }
            cache_put_data('user_settings-' . $member, null, 60);
        }
    }
} 

[serdominik]

a mio avviso c'è un po' di confusione nella funzione:
ci sono 3 foreach consegutivi prendo il primo e il terzo:

foreach ($data as $var => $val)
    {
        if ($val === '+')
            $data[$var] = $var . ' + 1';
        elseif ($val === '-')
            $data[$var] = $var . ' - 1';
    } 

l'altro

$setString = '';
    foreach ($data as $var => $val)
    {
        $setString .= "
            $var = $val,";
    } 

il terzo beh sembra almeno funzionare
ma il primo fai un'assegnazione alla variabile $data[$var] sia nell'if che nel elseif

sicuro che tutto questo sia ok? o c'è un po' di confusione?

[ShanReWeT]

Originariamente inviato da serdominik
a mio avviso c'è un po' di confusione nella funzione:
ci sono 3 foreach consegutivi prendo il primo e il terzo:

foreach ($data as $var => $val)
    {
        if ($val === '+')
            $data[$var] = $var . ' + 1';
        elseif ($val === '-')
            $data[$var] = $var . ' - 1';
    } 

l'altro

$setString = '';
    foreach ($data as $var => $val)
    {
        $setString .= "
            $var = $val,";
    } 

il terzo beh sembra almeno funzionare
ma il primo fai un'assegnazione alla variabile $data[$var] sia nell'if che nel elseif

sicuro che tutto questo sia ok? o c'è un po' di confusione?

il codice non l'ho fatto io.. è di SimpleMachinesForum dovrebbe essere corretto dato che non mi da nessun problema.

[serdominik]

Originariamente inviato da ShanReWeT
il codice non l'ho fatto io.. è di SimpleMachinesForum dovrebbe essere corretto dato che non mi da nessun problema.

allora guarda dovresti + provare a chiedere sui loro forum visto che è un cms/forum conviene + chiedere a loro che ci lavorano su quel software