View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002246EresseaREKRUTIEREpublic2016-10-07 14:332017-03-02 07:13
ReporterEnno Assigned ToEnno  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version3.9.2 
Target Version3.10.0Fixed in Version3.10.0 
Summary0002246: TEMP-Einheiten haben nicht rekrutiert
Description

... und keinen Fehler gemeldet bekommen!

Ich nehme an, dass mein Nachbar die 5 Personen in der Region bekommen hat. Usidor (dcs8) hat mehrere TEMP-Einheiten erstellt, die jede 1 Person rekrutieren sollten, aber das ist nicht passiert, und einen Fehler gemeldet hat der Server auch nicht. Das sollte so nicht sein.

Additional Information

Hat mit dem Bug direkt nichts zu tun, fiel mir aber auf:

In expandrecruiting ist immer noch Code für Zentauren drin (die rekrutieren offenbar aus den Pferden der Region). Pferdemenschen gibt es nicht (mehr), und der Code sollte weg.

/* centaurs: */
recruits = select_recruitment(&recruitorders, horse_recruiters, &orc_total);
TagsNo tags attached.
Factionufo
GameE2
Report994

Activities

Enno

Enno

2016-10-08 16:06

administrator   ~0006796

Einheiten, die überhaupt keine Personen rekrutieren können, bekommen keine Meldung. Meldungen werden in add_recruits erzeugt, und im Code von do_recruiting steht:

        if (number > 0) {
            add_recruits(u, number, req->qty);

Das ist wohl schon immer so.
Enno

Enno

2016-11-05 16:05

administrator   ~0006813

Wenn man das add_recruits vor das if steckt, sollte das eigentlich tun? Mal gucken.
Enno

Enno

2016-11-05 17:30

administrator   ~0006818

Es werden jetzt auch für TEMP-Einheiten Fehler gemeldet.

PR https://github.com/eressea/server/pull/600

Resultat: Einheit ga75 (ga75) in ☠ Danger Zone ☠ (0,0) rekrutiert 0 von 1 Personen. Einheit yx9d (yx9d) in ☠ Danger Zone ☠ (0,0) rekrutiert 0 von 1 Personen. Einheit 6sL4 (6sL4) in ☠ Danger Zone ☠ (0,0) rekrutiert 0 von 1 Personen.

Issue History

Date Modified Username Field Change
2016-10-07 14:33 Enno New Issue
2016-10-07 14:33 Enno Status new => assigned
2016-10-07 14:33 Enno Assigned To => Enno
2016-10-07 14:34 Enno Category General => REKRUTIERE
2016-10-08 16:06 Enno Note Added: 0006796
2016-11-05 16:05 Enno Note Added: 0006813
2016-11-05 17:30 Enno Status assigned => resolved
2016-11-05 17:30 Enno Resolution open => fixed
2016-11-05 17:30 Enno Fixed in Version => 3.10.0
2016-11-05 17:30 Enno Note Added: 0006818
2016-11-05 17:32 Enno Target Version => 3.10.0
2017-03-02 07:13 Enno Status resolved => closed