OnPlayerWeaponShot Problem

if(hittype == BULLET_HIT_TYPE_PLAYER_OBJECT)
{
if(amkaputtmachen[playerid] == 1)
{
for(new drug=0;drug<MAX_DROGEN;drug++)
{
if(IsPlayerInRangeOfPoint(playerid,3.5,DrogenInfo[drug][DrugX],DrogenInfo[drug][DrugY],DrogenInfo[drug][DrugZ]) && DrogenInfo[drug][druggepflanzt] == 1)
{
if(IsValidObject(hitid) && hitid == DrogenInfo[drug][drugobjekt])
{
DestroyObject(DrogenInfo[drug][drugobjekt]);
Delete3DTextLabel(DrugText[drug]);
format(query,sizeof(query),"DELETE FROM drogen WHERE drogenid = '%d'",DrogenInfo[drug][drogenid]);
mysql_function_query(MYSQLVerbindung,query,false,"","");
print("Aufruf bei Objekt getroffen!");
print("Aufruf für return bei ValidObject");
return true;
}
}
}
print("Ende for schleife");
return true;
}
} //hier hat noch eine Klammer gefehlt, vergessen zu kopieren nehme ich an.

Der return muss unter die IsValidObject(hitid) - Abfrage, da sonst die Schleife nach dem ersten Durchlauf beendet wird.
Wenn du es dort return'st dann wird der print ganz unten ("Ende for schleife") nicht mehr aufgerufen. Wenn der aufgerufen werden soll, musst du das "return true;" innerhalb der Schleife zu "break;" machen.

Zwar in Ordnung:

format(query,sizeof(query),"DELETE FROM drogen WHERE drogenid = '%d'",DrogenInfo[drug][drogenid]);

Allerdings sind die ' nicht nötig, wenn Du einen Integer ersetzt.

Kannst Du auch so schreiben:

format(query,sizeof(query),"DELETE FROM drogen WHERE drogenid = %d",DrogenInfo[drug][drogenid]);

Debugge es mal und poste dann was im Server Log steht (mit den Zeitstempeln bitte).
printf("playerid: %d", playerid);
if(hittype == BULLET_HIT_TYPE_OBJECT)
{
printf("amkaputtmachen: %d", amkaputtmachen[playerid]);
if(amkaputtmachen[playerid] == 1)
{
for(new drug=0;drug<MAX_DROGEN;drug++)
{
printf("drug: %d -> druggepflanzt: %d", drug, DrogenInfo[drug][druggepflanzt]);
if(IsPlayerInRangeOfPoint(playerid,3.5,DrogenInfo[drug][DrugX],DrogenInfo[drug][DrugY],DrogenInfo[drug][DrugZ]) && DrogenInfo[drug][druggepflanzt] == 1)
{
printf("In Range -> %d == %d (%d)", hitid, rogenInfo[drug][drugobjekt], IsValidObject(hitid));
if(IsValidObject(hitid) && hitid == DrogenInfo[drug][drugobjekt])
{
printf("Passt.");
DestroyObject(DrogenInfo[drug][drugobjekt]);
Delete3DTextLabel(DrugText[drug]);
format(query,sizeof(query),"DELETE FROM drogen WHERE drogenid = '%d'",DrogenInfo[drug][drogenid]);
mysql_function_query(MYSQLVerbindung,query,false,"","");
print("Aufruf bei Objekt getroffen!");
print("Aufruf für return bei ValidObject");
return true;
}
}
}
print("Ende for schleife");
return true;
}
}

Der Flammenwerfer funktioniert, zumindest laut Wiki.
Allerdings geht das Callback nur, wenn du den lagcompmode eingeschaltet hast. Hast du das aus?

Quelle: http://wiki.sa-mp.com/wiki/OnPlayerWeaponShot

Quelle: http://wiki.sa-mp.com/wiki/OnPlayerWeaponShot

Du hast aber nicht zufällig eine mehrzeilige Meldung im Compiler mit vielen Zahlen?

Ansonsten poste mal dein ganzes OnPlayerWeaponShot.

Ok, sehr komisch.

Setze den print mal vor das "new query...".
Dann gehe in den Server und probiere es mit verschiedenen Waffen. Wird das playerid dann mit irgendeiner Waffe geprintet?

Wenn nein: Hast du auch den 0.3z Server, oder nutzt du nur dessen Includes?

Vielleicht liegt es auch an dem Objekt das du gesetzt hast.
Ist es ein Objekt durch das man durchlaufen kann?

Ändere den print auch mal zu:
printf("playerid: %d | weaponid: %d | hittype: %d | hitid: %d | x:%f | y:%f | z:%f", playerid, weaponid, hittype, hitid, fX, fY, fZ);

Dann weißt du genau was passiert, wenn du schießt.

Ich denke nicht, denn du kannst durch diese ja problemlos durchlaufen, das heißt auch durchschießen, somit wird das Objekt eigentlich nicht getroffen.
Mit Pflanzen wird den Vorhaben nicht klappen, du musst vielleicht ein Objekt in die Pflanze setzen, welches du durchsichtig machst.