[Freeciv-Dev] [bug #13654] [patch] Help for Research Lab implies that University not required
Update of bug #13654 (project freeciv): Category:None = docs Status:None = Fixed Open/Closed:Open = Closed ___ Follow-up Comment #4: Attached a patch against S2_1 I committed this to TRUNK but not to S2_1. There were no errors in old text (we were just adding extra information), so I didn't want to change translatable string, invalidating all translations, in S2_1. ___ Reply to this item at: http://gna.org/bugs/?13654 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13654] [patch] Help for Research Lab implies that University not required
Follow-up Comment #3, bug #13654 (project freeciv): (Previous comment was me.) Attached a patch against S2_1 which updates Stock Exchange help (benefit is only 50%, not 100% as I said, as Bank by itself has no effect.) The bug name should be changed, but I don't think I can do it. (file #5950) ___ Additional Item Attachment: File name: stock-exchange-help.diff Size:0 KB ___ Reply to this item at: http://gna.org/bugs/?13654 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13654] [patch] Help for Research Lab implies that University not required
Follow-up Comment #1, bug #13654 (project freeciv): I think all this is intentional. You cannot build Research Lab without University, but if University (or Library) gets destroyed later, Research Lab is not rendered completely useless. ___ Reply to this item at: http://gna.org/bugs/?13654 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13654] [patch] Help for Research Lab implies that University not required
Follow-up Comment #2, bug #13654 (project freeciv): I hadn't thought of that. I had assumed that it was not possible to have the building without its prerequisites, but I don't see any restriction in the code enforcing that; AFAICT it's perfectly possible to raze, sabotage, or sell a prerequisite. So I retract my patch. In which case, the help for Stock Exchange ought in principle to be updated; as far as I can tell (untested), a Bank + Stock Exchange have a reduced effect (100% instead of 150%) even if the Marketplace is destroyed. ___ Reply to this item at: http://gna.org/bugs/?13654 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13654] [patch] Help for Research Lab implies that University not required
Update of bug #13654 (project freeciv): Assigned to:None = cazfi ___ Reply to this item at: http://gna.org/bugs/?13654 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13654] [patch] Help for Research Lab implies that University not required
URL: http://gna.org/bugs/?13654 Summary: [patch] Help for Research Lab implies that University not required Project: Freeciv Submitted by: jtn Submitted on: Monday 06/08/2009 at 21:55 Category: None Severity: 2 - Minor Priority: 5 - Normal Status: None Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Release: Operating System: None ___ Details: In the default ruleset, the help for the Research Lab says: Together with a Library, a Research Lab increases the science production of a city by 200%. However, Research Lab was made to depend on the presence of a University some time ago (r8580, PR#10364, apparently), so this statement is unhelpful. The attached patch removes the statement from the help. (I note that effects.ruleset still separates the effect with a Library -- effect_research_lab -- and University -- effect_research_lab_1. I haven't consolidated these.) ___ File Attachments: --- Date: Monday 06/08/2009 at 21:55 Name: research-lab-help.diff Size: 513B By: jtn http://gna.org/bugs/download.php?file_id=5920 ___ Reply to this item at: http://gna.org/bugs/?13654 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev