wäre das nicht ne gute Idee, so ne einfache Funktion noch hinzuzufügen.Würzi hat geschrieben:Nö was vorgefertigtes gibts da nicht
user_points aus der phpbb_users halt abfragen
Also $user->data['user_points']
Siehst du dann z.B. auch so in der download/file.php
irgendso etwas würde ich vorschlagen.Würzi hat geschrieben:Was wäre denn der Vorteil, wenn man das seperat z.b. als function substract_points_positiv als neue Funktion in die functions_points mit aufnimmt?
Ist sinnvoll um code zu sparen, wenn man eine Funktion öfter verwendet oder?
Code: Alles auswählen
/**
* Substract points from user, but only if points are positive
*/
function substract_points_condition($user_id, $amount)
{
global $db, $user;
// Select users current points
$sql_array = array(
'SELECT' => 'user_points',
'FROM' => array(
USERS_TABLE => 'u',
),
'WHERE' => 'user_id = ' . (int) $user_id,
);
$sql = $db->sql_build_query('SELECT', $sql_array);
$result = $db->sql_query($sql);
$user_points = $db->sql_fetchfield('user_points');
$db->sql_freeresult($result);
if ( ($user_points - $amount) <= 0 )
{
return false;
}
else
{
// Update the points
$data = array(
'user_points' => $user_points - $amount,
);
$sql = 'UPDATE ' . USERS_TABLE . '
SET ' . $db->sql_build_array('UPDATE', $data) . '
WHERE user_id = ' . (int) $user_id;
$db->sql_query($sql);
}
return;
}
Code: Alles auswählen
// Start Ultimate Points
'P_NAME' => $config['points_name'],
'U_LOCKED' => !$locked,
'U_BLOCKED' => !$blocked,
'USE_POINTS' => $config['points_enable'],
'USE_IMAGES_POINTS' => $points_config['images_memberlist_enable'],
'USE_BANK' => $points_config['bank_enable'],
'BANK_GOLD' => sprintf(number_format_points($bank_row['holding'])),
// End Ultimate Points
Jep und warscheinlich die "install.xml" anstatt die "contrib/Update_1.0.2_to_1.0.3/update.xml" abgearbeitet.
![[ externes Bild ]](http://www.wuerziworld.de/Smilies/fig/fig54.gif){kind=link}