Auto-queue doesn't play well with heroes.

[wowgetoffyourcellphone]

Current Behavior:

Having a hero in the auto queue turns AQ off when he's done training, probably because the hero has a train limit.

Expected behavior:

Units with train limit gets removed from the queue, allowing it to continue.

[phosit]

Is it intentional that the title is similar to #6278? :)

The underlying problem is likely the same and the solution/"behavior after fix" should be consistent.

[Stan]

Is it intentional that the title is similar to #6278? :)

Yep. Found it funny.

Doesn't have to be consistent that's up to the one fixing it.

[marder]

Replying to phosit:

Is it intentional that the title is similar to #6278? :)

The underlying problem is likely the same and the solution/"behavior after fix" should be consistent.

It's not 100% the same. From a quick look I think the hero thing could be simply fixed by removing this line ​https://github.com/0ad/0ad/blob/5651f6d93c7eb27d0d53ef4475713a30d4ba25e7/binaries/data/mods/public/simulation/components/ProductionQueue.js#L446

while the corrals require more thought. There, the queue should / cannot skip to the next item and would rather need to pause (as it does when one reaches the pop limit).

[Stan]

Maybe we could push a clickable notification that the production queue stopped.

[obelix]

This ticket seems to be redundant to #6669