Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002305Eressea[Alle Projekte] Generalöffentlich2017-03-11 14:59
ReporterSoltharBearbeitung durchEnno 
PrioritätnormalAuswirkungkleinerer FehlerReproduzierbarnicht getestet
Status geschlossenLösungerledigt 
Produktversion3.11.1 
Zielversion3.11.1Behoben in Version3.11.1 
Zusammenfassung0002305: Parteitarnung kaputt
Beschreibung

Im heute verschickten Testreport ist die Parteitarnung weg. Statt 1;Parteitarnung steht jetzt 1064694;Partei mit der richtigen Parteinummer. Zum Beispiel Einheit 2b7i.

Parteidrac
SpielE3
Report397

Eintrags-Beziehungen

verwandt mit 0002303 geschlossenEnno Als andere Partei getarnte Einheiten werden als Verräter markiert 
verwandt mit 0002309 geschlossenEnno Einheiten von Parteien, die HELFE PARTEITARNUNG setzen, zeigen keine Partei an. 

Notizen / Dateien

Solthar

Solthar

2017-03-07 09:41

Entwickler   ~0007073

Bin nicht sicher, ob das der gleiche Bug ist oder ein anderer: Bei einigen Einheiten ist keine Partei mehr im NR

  • Tet Tzopeh (j9tu), 1 Zwerg, hat: Amulett, Kettenhemd, 30 Phiolen,
    statt bisher
  • Tet Tzopeh (j9tu), Thorans Axtträger (d08a), 1 Zwerg, hat: Amulett,

Im CR erscheint die Partei korrekt: 899202;Partei

Enno

Enno

2017-03-07 19:31

Administrator   ~0007077

Ich glaube ich habe das mit der Parteitarnung raus (commit 29a8352b). Das Test-Framework für den CR, das ich am Wochenende gebaut habe, zahlt sich direkt aus.

Enno

Enno

2017-03-07 19:34

Administrator   ~0007078

@Solthar: Es gibt in E3 keine Partei d08a. Geht es hier um E2? Wenn ja, um welchen Report handelt es sich da? Bitte im Zweifel immer separate Bugs melden, ich finde das schon raus, wenn die zusammen gehören.

Enno

Enno

2017-03-07 20:14

Administrator   ~0007079

Antwort: Das ist E2, Report 1015, Partei 1wpy. Schauen wir mal.

Enno

Enno

2017-03-07 20:42

Administrator   ~0007080

Einheit 899202, Region 290911796. Die Einheit steht auf Schiff 1400525. Ich sehe den Fehler auch schon. Ich habe da etwas an get_otherfaction geändert.

Enno

Enno

2017-03-08 20:17

Administrator   ~0007082

Ich habe jetzt einen Test, der alle wichtigen Kombinationen von Tarnung testet, und bei dieser Kombination einen Fehler meldet.

Enno

Enno

2017-03-08 20:36

Administrator   ~0007084

https://github.com/eressea/server/pull/669

Solthar

Solthar

2017-03-09 23:28

Entwickler   ~0007092

Sorry, kann das sein, dass das im E2-Testreport vom 8.3. noch nicht behoben ist?

Ich habe hier
EINHEIT 1320096
"Träger";Name
808942;Partei
1;Parteitarnung

Also die Einheit saLc der Partei 808942 (hc6m), die aber nicht meiner Partei (1wpy) angehört. Somit dürfte ich die Partei gar nicht sehen.

Es ist spät, vielleicht sehe ich grade Gespenster. Ich schaue morgen noch mal drauf.

Enno

Enno

2017-03-10 03:03

Administrator   ~0007093

In der Kombination heißt das üblicherweise, dass die Einheit anonym ist, also TARNE PARTEI gemacht hat, aber die Partei Dir HELFE PARTEITARNUNG gesetzt hat. Deshalb weiß dein Report trotzdem, wer das ist. Was steht denn im NR, zum Vergleich?

Solthar

Solthar

2017-03-10 09:08

Entwickler   ~0007095

  • Träger (saLc), anonym, 100 Orks
Solthar

Solthar

2017-03-10 09:13

Entwickler   ~0007096

1;Parteitarnung heißt üblicherweise, dass ich sonst gar nichts sehe. Zumindest war das immer so. Es kann sein, dass die Partei h6cm mir HELFE Parteitarnung gesetzt hat. Das hat aber bisher keinen Unterschied gemacht. Was ein bisschen unlogisch ist. Wenn es also daran läge, wäre das ein neues Verhalten, dass ich aber eigentlich für konsequenter halten würde. Dann müsste das aber auch im NR sichtbar werden.

Enno

Enno

2017-03-10 09:29

Administrator   ~0007097

Seufz. Ich bin bei meinen Tests für den CR davon ausgegangen, dass HELFE PARTEITARNUNG bedeutet, dass der Alliierte das selbe sieht, was der Besitzer sieht. Also auch, wem eine anonyme Einheit wirklich gehört. Wenn im NR bei Dir "anonym" steht, dann hat der HELFE-Status aber wohl keine Auswirkung auf Anonymisierung? Warum? Die Regeln sagen dazu: "Parteien, die von der Partei oder Gruppe der Einheit HELFE xyz PARTEITARNUNG bekommen, können die wahre Parteizugehörigkeit der Einheit sehen."

Ich sollte mal nachgucken: Ist es wirklich wahr, dass die Partei Dir HELFE gesetzt hat?

Enno

Enno

2017-03-10 15:35

Administrator   ~0007098

Die Partei hat Dir HELFE gesetzt, und deswegen zeigt der Code die Partei im CR an. Da wird in der aktuellen Version nicht zwischen dem Besitzer und Alliierten unterschieden.
Der NR verhält sich anders, das stimmt. Ich bin ja der Meinung, der NR ist falsch, aber das war wohl schon immer so?

Enno

Enno

2017-03-10 23:29

Administrator   ~0007103

Wir lassen das erst einmal so, wie es ist, mit der zusaetzlichen Information im CR.

Enno

Enno

2017-03-11 14:59

Administrator   ~0007106

Alle Bugs sind gefixt, Version 3.11.1 ist ausgerollt.

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2017-03-07 09:12 Solthar Neuer Eintrag
2017-03-07 09:15 Enno Bearbeitung durch => Enno
2017-03-07 09:15 Enno Status neu => zugewiesen
2017-03-07 09:16 Enno Zielversion => 3.11.1
2017-03-07 09:16 Solthar Beziehung hinzugefügt verwandt mit 0002303
2017-03-07 09:41 Solthar Notiz hinzugefügt: 0007073
2017-03-07 19:31 Enno Notiz hinzugefügt: 0007077
2017-03-07 19:34 Enno Notiz hinzugefügt: 0007078
2017-03-07 20:14 Enno Notiz hinzugefügt: 0007079
2017-03-07 20:42 Enno Notiz hinzugefügt: 0007080
2017-03-08 18:14 Enno Beziehung hinzugefügt verwandt mit 0002309
2017-03-08 20:17 Enno Notiz hinzugefügt: 0007082
2017-03-08 20:36 Enno Status zugewiesen => erledigt
2017-03-08 20:36 Enno Lösung offen => erledigt
2017-03-08 20:36 Enno Behoben in Version => 3.11.1
2017-03-08 20:36 Enno Notiz hinzugefügt: 0007084
2017-03-09 23:28 Solthar Status erledigt => Rückmeldung
2017-03-09 23:28 Solthar Lösung erledigt => wiedereröffnet
2017-03-09 23:28 Solthar Notiz hinzugefügt: 0007092
2017-03-10 03:03 Enno Notiz hinzugefügt: 0007093
2017-03-10 09:08 Solthar Notiz hinzugefügt: 0007095
2017-03-10 09:08 Solthar Status Rückmeldung => zugewiesen
2017-03-10 09:13 Solthar Notiz hinzugefügt: 0007096
2017-03-10 09:29 Enno Notiz hinzugefügt: 0007097
2017-03-10 15:35 Enno Notiz hinzugefügt: 0007098
2017-03-10 23:29 Enno Status zugewiesen => erledigt
2017-03-10 23:29 Enno Lösung wiedereröffnet => erledigt
2017-03-10 23:29 Enno Notiz hinzugefügt: 0007103
2017-03-11 14:59 Enno Notiz hinzugefügt: 0007106
2017-03-11 14:59 Enno Status erledigt => geschlossen