ticker error mit aktueller population formel

[Agatho]

Salut also ich hab die aktuelle Bevölkerungsformel aus den Regeln genommen und bekomme nun beim Ticker folgenden Fehler

Code: Alles auswählen

2005-11-22 09:46:27 ticker/ticker.c:254: db_query: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'r_bier = GREATEST(LEAST(r_bier + r_bier_delta, 3000+2000*b_lagerraum), 0),r_hanf' at line 1: UPDATE Cave SET r_population_delta = (LEAST(r_nahrung-SIGN(r_population),(b_partyraum*6+b_schlafplatz/10+2)*(1+population_factor)+population_bonus)) * IF((LEAST(r_nahrung-SIGN(r_population),(b_partyraum*6+b_schlafplatz/10+2)*(1+population_factor)+population_bonus)) <= 0, 1

Der Fehler scheint erst nach dem IF zu beginnen. wenn man nur das UPDATE nimmt geht das also muss die Formel an sich ja richtig sein.
Ich hoffe hier kann mir jemand helfen

MySql Version ist MySQL 5.0.15
Source Version r5b 17.11

Danke und liebe Grüsse

Agatho

[Hanschi]

ich zähle 11 mal Klammer auf und nur 10x Klammer zu?

ist nur der Versuch einer Lösung ohne obige Zeile richtig zu verstehen

[Tenobaal]

[..] IF((LEAST(r_nahrung-SIGN(r_population)),(b_partyraum*6+b_schlafplatz/10+2)*(1+population_factor)+population_bonus)) <= 0, 1[/code]
[..]

Diese Klammer scheint gefehlt zu haben, so dass das Komma noch zu der Rechnung und nicht zu LEAST gezählt wurde.

[Firak]

ich zähle 11 mal Klammer auf und nur 10x Klammer zu?

ist nur der Versuch einer Lösung ohne obige Zeile richtig zu verstehen

Das SQL-Statement ist (in der Ausgabe) am Ende abgeschnitten, so daß Klammernzählen hier keinen Sinn hat...

Entweder DEBUG_SQL im Ticker einschalten (vermutlich muß man dazu den Ticker neu compilieren) oder mal von Hand die Produktions- und/oder MaxStorage-Formeln in den Spielregeln durchgehen, würde ich sagen.

[Agatho]

Naja war nur ein Klammerfehler nobody 's perfect. Naja ein weiters Problem ist nun, daß man nicht seine in nachrichten sieht. es wird zwar gesagt das man eine neue Nachricht hat und auch das Symbol erscheint allerdings ist die Listendarstellung leer. Out Nachrichten sieht man allerdings ganz normal und ein fehler wird auch nicht ausgegeben.
Der fehler muss in der message.inc.php liegen und zwar in diesem codeabschnitt.

Code: Alles auswählen

function messages_getIncomingMessages($offset, $row_count){
  global $db, $params, $config;

  // get message classes
  $uaMessageClass = MessageClass::getMessageClasses();


  $nachrichten = array();

  // get announcements
  $query =  'SELECT ' .
            'm.messageID, ' .
            'p.name, ' .
            'm.messageClass, ' .
            'm.messageSubject AS betreff, ' .
            'm.messageTime ' .

            'FROM Message m ' .
            'LEFT JOIN Player p ' .
            'ON p.playerID = m.senderID ' .

            'WHERE ' .
            'messageClass = 1001 ' .
            'ORDER BY m.messageTime DESC, m.messageID DESC';

  if (!($dbresult = $db->query($query)))
    return array();

  while($row = $dbresult->nextRow(MYSQL_ASSOC)){
    $row['absender_empfaenger'] = empty($row['name']) ? _MSG_SYSTEM : $row['name'];
    $t = $row['messageTime'];
    $row['datum'] = $t{6}.$t{7}  .".".
                    $t{4}.$t{5}  .".".
                    $t{2}.$t{3}  ." ".
                    $t{8}.$t{9}  .":".
                    $t{10}.$t{11}.":".
                    $t{12}.$t{13};
    $row['nachrichtenart'] = $uaMessageClass[$row['messageClass']];
    $row['linkparams'] = '?modus=messagesdetail&messageID=' . $row['messageID'] . '&box=' . BOX_INCOMING;
    $nachrichten[] = $row;
  }

  // get user messages
  $query = 'SELECT m.messageID, m.flag, p.name, m.messageClass, m.messageSubject AS betreff, m.messageTime, SIGN(m.read) as `read` ' .
           'FROM Message m ' .
           'LEFT JOIN Player p ' .
           'ON p.playerID = m.senderID ' .
           'WHERE ' .
           'recipientID = ' . $params->SESSION->player->playerID . ' ' .
           'AND recipientDeleted != 1 ' .
           'ORDER BY m.messageTime DESC, m.messageID DESC '.
           'LIMIT ' . intval($offset) . ',' . intval($row_count);

  if (!($dbresult = $db->query($query)))
    return array();

  while($row = $dbresult->nextRow(MYSQL_ASSOC)){
    $row['absender_empfaenger'] = empty($row['name']) ? _MSG_SYSTEM : $row['name'];
    $t = $row['messageTime'];
    $row['datum'] = $t{6}.$t{7}  .".".
                    $t{4}.$t{5}  .".".
                    $t{2}.$t{3}  ." ".
                    $t{8}.$t{9}  .":".
                    $t{10}.$t{11}.":".
                    $t{12}.$t{13};
    $row['nachrichtenart'] = $uaMessageClass[$row['messageClass']];
    $row['linkparams'] = '?modus=messagesdetail&messageID=' . $row['messageID'] . '&box=' . BOX_INCOMING;
    $row[($row['flag'] ? 'FLAGGED' : 'UNFLAGGED') . '/id'] = $row['messageID'];
    $nachrichten[] = $row;
  }
  return $nachrichten;
}

vielleicht sieht da ja einer mehr als ich.
wenn man allerdings das mit der outgoing funktion ersetzt (und anpasst) geht das incoming ist allerdings keine adäquarte lösung