Scenario does not continue in empire 2 campaign

[Leif Sandstede]

If you upgrade the Marmormine before fulfilling the other conditions the Barbarians will not be revealed, unless you build another, not upgraded, mine.

[Hans Joachim Desserud (hjd)]

Hi Leif, and welcome to Launchpad!

I played a short test game right now, and experienced the same issue. I am not sure why this happens though. I thought it would already have registered that a marble mine has been built, when that objective is completed. However based on this, it seems like it checks buildings sometime later as well to see how far along the player is.

[Leif Sandstede (lcsand)]

I know why it happens, it continually checks if certain certain conditions are fulfilled.
In this case it checks if at least one of each mine is built. Problem is that it only checks for not upgraded marmormines. Upgraded ones are not counted. I will try to commit a patch that fixes this, once I figure out how to do that. I can already say that I like git better than bazar.

[Hans Joachim Desserud (hjd)]

Sounds great. :) I'm assigning this bug to you, to let others know someone are working on it.

Uploading your changes to Launchpad should be fairly straightforward. If you haven't seen them yet, you might want to take a look at https://help.launchpad.net/YourAccount/CreatingAnSSHKeyPair and https://help.launchpad.net/Code/UploadingABranch , then make a merge proposal for your branch and someone will take a look at it.

[Leif Sandstede (lcsand)]

OK fix is uploaded, hope I did this correctly.

[Hans Joachim Desserud (hjd)]

Seems right to me. Just please don't change the bug status to fix committed before it has been merged to trunk (though it looks like you realized that on your own).

Thanks for providing a fix, now we just have to wait for someone to take a look at it.

[SirVer (sirver)]

Merged in 5915. See comment for other bug 741142. Thanks for contributing!

[SirVer (sirver)]

Released in build17-rc1.