Skip to content
This repository has been archived by the owner on Apr 17, 2022. It is now read-only.

flashlight tanks become pulse laser tanks when pulse laser becomes researched #2877

Closed
wzdev-ci opened this issue Sep 4, 2011 · 26 comments
Closed

Comments

@wzdev-ci
Copy link
Contributor

wzdev-ci commented Sep 4, 2011

resolution_closed type_bug | by Reg312


flashlight tanks become pulse laser tanks when pulse laser becomes researched
how reproduce bug:

  1. make 10 flashlight python tracks
  2. research pulse laser
  3. see u have 10 pulse laser tanks labeled "flashlight..."
    damage and animation like pulse laser

Issue migrated from trac:2877 at 2022-04-16 08:35:14 -0700

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 4, 2011

Kreuvf changed status from new to closed

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 4, 2011

Kreuvf changed resolution from `` to invalid

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 4, 2011

Kreuvf commented


Not a bug. This is on purpose. The only thing one could criticise here is that the unit name stays the same.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 4, 2011

effigy commented


I know the turret changes from Flashlight from Pulse in the design menu, but I never noticed units that had already been produced being changed.

Also, in the design menu if you change an existing Flashlight design to Pulse the name Flashlight will be retained in the unit name.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 4, 2011

Reg312 commented


its strange purpose, anyway lasers never get used in multiplayer games like most other T3 weapons

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 4, 2011

dak180 changed status from closed to reopened

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 4, 2011

dak180 changed resolution from invalid to ``

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 4, 2011

dak180 commented


Already built units should not spontaneously change weapon types in the field.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 5, 2011

Kreuvf commented


Replying to Warzone2100/old-trac-import#2877 (comment:2):

Also, in the design menu if you change an existing Flashlight design to Pulse the name Flashlight will be retained in the unit name.
That's because once the unit name is set, it will stay. You can name your units using own names (e. g. "effigy's Hilarious Laser Tank pew pew") and that is why WZ does not even try to change the name, because there is no guarantee that people do not use own names.

inb4: people suggesting turning off the possibility to give own names to units... -.-°

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 5, 2011

Kreuvf commented


Replying to Warzone2100/old-trac-import#2877 (comment:4):

Already built units should not spontaneously change weapon types in the field.
No reason given. And please don't insist on "logic". Weapon upgrades should not work then as well, but magically every unit no matter how far away from home base receives the new upgrade (more damage, faster reloading, more armour, whatever!). It's a game and it's special for the things that "surprise" you, the things it does differently than other games and it is that quality which led to all the efforts that finally resulted in the liberation of the game.

Commander turrets are replaced in the exact same manner (especially since you would not be able to tell different commander turrents apart).

People should be informed in-game about what which research topic does. An in-game tech-tree or some other thing that allows people to assess the complexity of WZ's research would be the way to go instead of altering the game beyond a point where it behaves like any other game.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 5, 2011

anonymous commented


Maybe Flashlight and Pulse should be either separated then or unified, so the upgrade path is not confusing.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 5, 2011

Reg312 commented


My reasons why it's "bug":

  1. WZ should have identical research logic. Game logic should be universal.
    Why heavy MG tanks don't become assault gun tanks?
  2. commander turrets do not count beacuse its not weapon and model&animmation not changing.
  3. removing such features making easier to adjust game balance
  4. this feature with pulse laser don't has reason why it happening

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Sep 6, 2011

Kreuvf commented


Replying to Warzone2100/old-trac-import#2877 (comment:8):

My reasons why it's "bug":

  1. WZ should have identical research logic. Game logic should be universal.
    ???

Why heavy MG tanks don't become assault gun tanks?
The original developers might know.

  1. commander turrets do not count beacuse its not weapon and model&animmation not changing.
    Actually, technically it is a weapon. Not sure if this has been changed, but Ripple Rockets are/were replaced by another long range rocket as well.
  1. removing such features making easier to adjust game balance
    ??? This is no valid reason. Using this you could justify removal of almost anything just for the sake of "making things easier".
  1. this feature with pulse laser don't has reason why it happening
    What I said: the game does not make this clear to the user and that is something that has to be changed. You should know before researching what the research does. Educating the user about his/her options is something WZ is admittedly bad at.

@wzdev-ci
Copy link
Contributor Author

Zarel commented


Yes, Warzone used to have automatic research upgrades; Flashlight to Pulse was one. Ripple to Archie was another.

If I remember correctly, these were removed a while ago because they were confusing to players, so if it still occurs, it's a bug. My memory may be faulty, however.

@wzdev-ci
Copy link
Contributor Author

Zarel commented


Oh, wait, this might be campaign. We never touched the automatic research upgrade behavior in campaign, so if it happens in campaign, that's just expected behavior, right down to the tanks still being called "Flashlight" even after the upgrade.

@wzdev-ci
Copy link
Contributor Author

Reg312 commented


it happens in skirmish

@wzdev-ci
Copy link
Contributor Author

Zarel commented


80% sure it's some sort of bug, then.

@wzdev-ci
Copy link
Contributor Author

Iluvalar commented


This is supposed to be a range upgrade for the flashlight. It's not a bug.

If it was, the replaced part could be erased at
data/mp/stats/research/multiplayer/resultcomponent.txt
line 69 and 70.

@wzdev-ci
Copy link
Contributor Author

Iluvalar uploaded file flashlight.patch (1.2 KiB)

@wzdev-ci
Copy link
Contributor Author

Shadow Wolf TJC changed _comment0 which not transferred by tractive

@wzdev-ci
Copy link
Contributor Author

Shadow Wolf TJC commented


Indeed. Just as Iluvalar said, this "bug" can be fixed by removing the "obsolete component" information from various entries in resultcomponent.txt. The components will still disappear from the design screen if they're included in redcomponents.txt.

Although I only recently discovered this "feature" today, I kind of like it. It's a nice little feature that modders can use to automatically replace older weapons with newer ones, and I feel that it shouldn't be changed at all. Moreover, it looks like the "bug" was fixed by modifying resultcomponent.txt appropriately, so I guess that the ticket can be closed now?

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Apr 25, 2012

Iluvalar commented


Replying to Warzone2100/old-trac-import#2877 (comment:15):

so I guess that the ticket can be closed now?
No it can't because as weird as it look, they never applied that patch to fix the bug. ­¬.¬

@wzdev-ci
Copy link
Contributor Author

Shadow Wolf TJC commented


Ah, I see. I checked resultcomponent.txt yesterday, and I didn't see that Pulse Laser replaced Flashlight. I just checked it again, and the problem is indeed still there. Sorry about that.

@wzdev-ci
Copy link
Contributor Author

Berserk Cyborg changed status from reopened to closed

@wzdev-ci
Copy link
Contributor Author

Berserk Cyborg changed resolution from `` to closed

@wzdev-ci
Copy link
Contributor Author

Berserk Cyborg commented


This isn't the case anymore. If this behavior is desired, feel free to discuss it in the balance forum.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant