Das liegt daran, dass deine Variable vehicleid immer 0 ist. Du setzt da keinen Wert.
Deshalb wenn GetVehicleModel(0) aufgerufen wird und da 0 keine gültige vehicleid ist, wird hier auch immer 0 zurückgegeben und 0-400 = -400 und dann greifst du auf diesen index zu, der natürlich nicht existiert.
Also vehicleid einen Wert zuweisen und sicher sein, dass das Auto auch existiert 
Interessant wäre die vollständige Debug Meldung...
Da steht ja nicht einmal ein Fehler..
Schau mal hier rein: Schneller & strukturierter skripten
Das könnte dir helfen, dann kannst du dir einiges schenken, wie mehrere Strings, die isLoggedIn Abfrage in jedem Befehl und eine einzelne Auflistung, da solltest du eine Schleife verwenden.
Wie zum teufel, hast du modelid deklariert bzw wie verwendest du das da, so wird das nicht funktionieren.
Und zu deiner Frage, du kannst dann einfach listitem verwenden und dann weißt du, auf welches Fahrzeug er geklickt hat, nämlich:
PrivateVehicle[playerid][listitem][modelid]
Du machst dir das viel zu kompliziert.
Die innere Schleife mit dem mInfo ist komplett unnötig, da du ja durch GetVehicleModel doch schon weißt, um welches Model es sich handelt.
public tankLostTimer()
{
for(new i=GetVehiclePoolSize(); i!=0; i--)
{
if(!motor[i]) continue;
if(tank[i] <= 0)
{
tank[i] = motor[i] = 0;
SetVehicleParamsEx(i, 0, 0, 0, 0, 0, 0, 0);
}
else tank[i] -= cInfo[i][lpm] / 60;
}
for(new i=GetPlayerPoolSize(),v,m,string[24]; i!=-1; i--)
{
if(IsPlayerNPC(i) || !IsPlayerInAnyVehicle(i)) continue;
v = GetPlayerVehicleID(i), m = GetVehicleModel(v);
format(string,sizeof(string),"%.02f / %.02f",tank[v],mInfo[m][maxTank]);
PlayerTextDrawSetString(i, Tacho_TD[i][15], string);
}
return 1;
}Ich glaube so macht das mehr Sinn.
So brauchst du auch keine Schleifen schachteln..weil ja sonst deine Laufzeit extremst wäre..du müsstest ja jedes mal MAX_VEHICLES*MAX_VEHICLES*MAX_PLAYERS durchlaufen...das ist ja Irrsinn.
Nimm das ganze Skript und pack es in Notepad++.
Dann klicke oben bei Kodierung Konvertiere zu ANSI.
Und dann klick oben auf UTF-8 und fertig, dann sollte das wieder normal sein 
so gibt es ein Fehler
Weil das Strings sind, dafür musst du [wiki]strcmp[/wiki] benutzen
if(strcmp(inputtext, Spieler[playerid][pPersoCode]) != 0)
Wichtig ist noch, dass weder inputtext noch Spieler[playerid][pPersoCode] leer sind
Hab ich doch geschrieben, einfach in der Funktion menuIsActive ganz nach oben.
Und wie auch schon erwähnt, kannst du echo benutzen, wenn dir das zu doof ist mit der Konsole.
bei deiner funktion kommt das in den logs
Du hast das aber unter das mysql_format gepackt oder? 
Zu dem anderen, wie gesagt, mysql_log: https://wiki.sa-mp.com/wiki/MySQL/R33#mysql_log
Das sagt eigentlich, dass die Download.php Seite gar nicht geladen werden soll.
Und schreib mal in die Funktion menuIsActive :
echo '<script>console.log("'.$filename.'", "'. basename($_SERVER['SCRIPT_FILENAME']).'");</script>';
Kannst aber eben auch wie gesagt echo benutzen und kannst im Quellcode dann schauen, was dir ausgegeben wird.
Dann kannst du F12 drücken und in der Console, sollte dann was stehen wie downloads.php
Ich weiß nicht was du da tun willst. Weil das die() sorgt dafür, dass die Seite aufhört zu laden.
Also die ist immer Leer.
damals hat es auch funktioniert mit der länge des strings
Dann printe dir doch einfach den String, dann findest du es doch raus:
printf("%s | %d",DB_Query, strlen(DB_Query));
Der Error Log ist der mysql_log.
Wenn du diesen aktiviert hast mit mysql_log dann werden dir dort alle mysql Aufrufe angezeigt und eben auch Errors oder Warnings.
Kann es sein, dass du anstatt dem echo in Zeile 3 ein if schreiben wolltest?
Dann sollte es eigentlich gehen.
Ansonsten einfach mal debuggen mit echo
new DB_Query[225];
Ich mutmaße einfach mal, dass der Query String zu klein ist.
Ich empfehle dir hier mal auf 400 zu gehen.
Durch ein einfaches print, oder den Error log findest du sowas einfach raus
Kriege das Irgendwie nicht hin kann das auch nicht
Naaa soo schwer ist das ganze ja nicht 
Du movst einfach das TextDraw per Menü nach ganz oben Links und dann kannst du ja die size ändern und hälst Leertaste gedrückt und schiebst es nach rechts unten.
Am Ende passt du die Farbe an.
0x FF FF FF 88
FF = R
FF = G
FF = B
88 = A
R = Red = Rot
G = Green = Grün
B = Blue = Blau
A = Alpha = Transparenz (je niedriger, desto durchsichtiger)
Ich könnte dir natürlich den Code geben, du könntest auch theoretisch die Koordinaten raten...denn oben Links ist ja bekanntlich 0,0 und dann musst du nur die rechts Unten liegende Koordinate abschätzen (bzw da gibt es einen genauen Wert).
Allerdings übs doch einfach mal für andere Textdraws, das könnte doch durchaus sehr praktisch sein, wenn du damit umgehen kannst
So schwer ist das Menü nicht zu bedienen, viel Glück!
Leute...vehicleid != vehiclemodel.
Ich glaube ihr wollt:
GetVehicleModel(GetPlayerVehicleID (playerid))
Edit: Kann man sich irgendwie wärend dem Spiel aus dem RCON ausloggen?
https://wiki.sa-mp.com/wiki/RCON#RCON_Commands
Also nein
Um mich nochmal kurz zu erklären, ich wollte auf dieses Zitat hinaus:
Entwickelt werden muss ein kleiner und überschaulicher Mode der eigenständiges Roleplay unterstützen soll
Ich finde dem eben nicht so. Denn wie ich es bereits erklärt hatte, damit jegliche Funktionalitäten für jede Fraktion etc erfüllt werden, muss man diese Skripten, selbst wenn man dann andere Abfragen hat, ob man zu der jeweiligen Fraktion gehört oder nicht.
Das bedeutet dennoch, dass man erstmal ein komplettes Skript braucht.
Und das ist nicht mal eben auf die Schnelle geschrieben.
Wie dem auch sei, trotzdem viel Erfolg weiterhin, bin mal gespannt, wie es am Ende umgesetzt wird.
PS: Ein Forum sollte immer ein Impressum besitzen. Gerade jetzt in Zeiten der neuen DSGVO sind da viele komische Leute unterwegs, die einen Orgasmus bekommen, sobald die jemanden abgemahnt haben. Also sei da vorsichtig.
Wie stellst du dir das vor?
Greasemonkey ist ein Browser Addon, mit dem man auf bestimmten Seiten Javascript ausführen kann.
Aber das ist natürlich Client-Seitig...wenn du sowas für ein Forum willst, musst du da das Addon schreiben und nicht in deinem eigenen Browser
Du kannst das eigentlich garnicht umwandeln.
Natürlich kann man das umwandeln, das sind doch schließlich nur Sekunden.
Gibt da eine Funktion die heißt mktime kannste ja mal googeln die wirste bestimmt irgendwo finden.
Oder wenn du schwere Geschütze auffahren magst: https://forum.sa-mp.com/showthread.php?t=294054
Lass die Schleife bei createshop weg
Dadurch wird das 200x erstellt, die brauchst du nicht.
