Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add tele_sight to itemgroups #34755

Merged
merged 1 commit into from
Oct 15, 2019
Merged

add tele_sight to itemgroups #34755

merged 1 commit into from
Oct 15, 2019

Conversation

snipercup
Copy link
Contributor

Summary

SUMMARY: Content "add telescopic sight to itemgroups"

Purpose of change

tele_sight can only be obtained trough crafting. This PR fixes that.

Describe the solution

Add tele_sight to the same itemgroups as improve_sights:

gunmod_common -> improve_sights(60)
military -> improve_sights(10)
drugdealer -> improve_sights(10)
map_extra_drugdeal -> drugdealer(75) -> improve_sights(10)
npc_misc -> improve_sights(1)
NC_JUNK_SHOPKEEP_misc -> improve_sights(20)
NC_EVAC_SHOPKEEP_misc -> improve_sights(20)

Describe alternatives you've considered

Leaving it as-is

Testing

image

Additional context

The NC_EVAC_SHOPKEEP_misc is a little weird I think because the result seems disproportionate and I think it is cause by allowing it to spawn items and itemgroups, which will give items such as the tele_sight a bigger spawn chance then items from itemgroups in that same itemgroup. Maybe something for another PR.

@tenmillimaster
Copy link
Member

So much no. The handmade telescope is a simple refractor/gallilean or similar telescope made of ~2 elements. It isn't even close to the same quality as a cheap tasco or chineseium scope.

If it spawns, it should only be alongside handmade survivor equipment.

Usable scopes like this could even be magnifying, but also inverting, in design, and they're far from ideal.

An alternative would be to add this style of telescope to the game, and have the handmade scope use it as a simpler recipe.

@snipercup
Copy link
Contributor Author

Nowhere in the item's description does it imply it is hand-made even though you can make it by hand.

@tenmillimaster
Copy link
Member

tenmillimaster commented Oct 15, 2019

Reference #23536, when they were introduced. Making pre-cata scopes was discussed; the sights that were able to be made are not those.

@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON Items / Item Actions / Item Qualities Items and how they work and interact Spawn Creatures, items, vehicles, locations appearing on map labels Oct 15, 2019
@ZhilkinSerg ZhilkinSerg merged commit cf395a3 into CleverRaven:master Oct 15, 2019
@ZhilkinSerg
Copy link
Contributor

ZhilkinSerg commented Oct 15, 2019

I've reverted this pull request, so it can be discussed further.

@snipercup
Copy link
Contributor Author

I can agree on items crafted by a survivor not being allowed to spawn in the same group as pre cataclysm items. I will cancel this pr. Would you agree that the description of the tele sight should be changed to reflect the fact it is no more then a crafted telescope with a crossheir?

@snipercup snipercup deleted the add-tele_sight-to-itemgroups branch October 15, 2019 16:26
@tenmillimaster
Copy link
Member

Yeah, that sounds reasonable. The items showing up in post cata shops is a good addition imo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Items / Item Actions / Item Qualities Items and how they work and interact [JSON] Changes (can be) made in JSON Spawn Creatures, items, vehicles, locations appearing on map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants