Claim munchies sitting on tables on farmers market

Kurz zur Info: Pull Request ist erstellt.

Mein Zweitaccount hat wieder was hervorgebracht, daher habe ich den Pull Request erst mal zurückgenommen. Neuer Code wird gerade getestet.

Die JSON Daten sind anders formatiert wenn man nur einen Tisch hat. Weiterhin sind diese nicht einheitlich formatiert.

Ein freigeschalteter Tisch ergibt mehrere Arrays mit Daten:
[php](...)
"tables": [
{
"id": 1,
"chairs": {
"1": [],
"2": [],
"3": [],
"4": []
}
},
{
"chairs": {
"1": [],
"2": [],
"3": [],
"4": []
},
"id": 2,
"block": 1,
"cost": 2500
},
(...)[/php]

Drei Tische ergeben die Objekte "0", "1", usw mit einem Array an Daten plus etwas Durcheinander bei den ID Attributen. Mal ist diese als Zahl angegeben, danach als Text.
[php](...)
"tables": {
"0": {
"id": 1,
"chairs": {
"1": [],
"2": [],
"3": [],
"4": []
}
},
"1": {
"id": "2",
"chairs": {
"1": [],
"2": [],
"3": [],
"4": []
}
},
(...)[/php]

Sobal dich etwas habe was auch bei meinem Level 12 Account funktioniert stelle ich es wieder ein.

Im Grunde ist die Änderung seit gestern aktiv (nur noch nicht commited), ich wollte nur noch die Nacht heute abwarten, wenn nochmals Munchies verarbeitet wurden.

Pull Request: https://github.com/HackerHarry/mffbashbot/pull/9

Manchmal hasse ich mich ja wenn ich mir Dinge nochmals ansehe

Den Pull Request habe ich kurz in meiner Mittagspause eingestellt.

Jetzt fällt mir noch auf, dass man die Zeile mit der Ermittlung der JSONDATATYPE Variabel auch außerhalb der FOR Schleife setzen kann, dann wird die Operation nicht unnötigerweise 5 mal aufgerufen Der Datatype ändert sich ja nicht innerhalb der Schleife.

Also so
[php](...)
echo "Checking for munchies sitting at tables..."
# JSON data uses different data types when you bought further tables on food world
JSONDATATYPE=$($JQBIN '.datablock.tables | type' $FARMDATAFILE)
for TABLE in 0 1 2 3 4; do
# Munchies on unleased tables can still be claimed, do not skip TABLE loop
for CHAIR in 1 2; do
(...)[/php]

@Harry: Es gibt die Möglichkeit Merges (Was ein Pull Request im Grunde ja ist) ohne Commit durchzuführen. Dann könntest Du das vor dem Commit noch anpassen. Ansonsten baue ich das bei nächster Gelegenheit mit ein - prinzipiell läuft es ja.

Das ist kein Problem, ich werde das so machen dass Du nur einen Pull Request mit allen aktuellen Änderungen hast.