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

[Request]Ability to specify license requirement overrides on a per-outfitter basis #7129

Open
Zitchas opened this issue Aug 14, 2022 · 9 comments
Labels
enhancement A suggestion for new content or functionality that requires code changes

Comments

@Zitchas
Copy link
Member

Zitchas commented Aug 14, 2022

Problem Description

There are situations when an outfit may be restricted by a license in one location, but not in another. For example, the Korath outfits that the Remnant eventually include in their outfitter will likely eventually have a Korath license restriction once the Exiles have an outfitter somewhere that the player may be able to visit. Likewise, there are plenty of opportunities for pirates or unfettered or other law-breakers in various factions to potentially sell stuff that the main source of which would prefer to remain restricted to licensed individuals.

Related Issue Links

I'm pretty sure more relevant license stuff has been mentioned, but I can't find it at the moment.

Desired Solution

I would like a way to specify, in the header of the outfitter description, a way to specify that additional licenses are required and/or that existing license requirements are cleared (or combinations thereof) for outfits contained within that outfitter list but only when accessing those outfits through that outfitter.

outfitter "Name of outfitter list"
	licenses
		remove <all>, <specified license>
		add <specified license>
	... list of outfits

Alternative Approaches

  1. Ability to specify in each outfit that it has different license requirements in a specified outfitter.
  2. Create duplicate outfits for each outfitter with different license requirements. Ex. "Remnant Salvaged Small Heat Shunt"

Additional Context

This would be used immediately by the Remnant and Korath; and strong potential for several other storyline areas.

@Zitchas Zitchas added the enhancement A suggestion for new content or functionality that requires code changes label Aug 14, 2022
@warp-core
Copy link
Contributor

If this is done, I think it would be better if any changes to normal licensing within the outfitter is set on a per outfit basis, as a child of each outfit entry.

@Zitchas
Copy link
Member Author

Zitchas commented Aug 14, 2022

So each outfit would have a sequence of sorts:

outfit "Korath Loot"
	license Korath
	outfitter "Remant Salvage"
		remove license Korath
		add license Remnant
	...

That sort of style of thing?

@warp-core
Copy link
Contributor

No. In the outfitter definition, under each outfit entry.

outfitter "Whatever Basics"
	"Very Good Outfit"
		<license>

It may also be worth refactoring licenses to be their own class that Ships and Outfits have.

@Zitchas
Copy link
Member Author

Zitchas commented Aug 14, 2022

Not quite as streamlined, but I can see the utility of doing it that way. That would work.

@EjoThims
Copy link
Member

I find Warp's proposed version much simpler to parse, personally.

For each outfitter this item appears in, specify what, if any, license is needed to purchase it from there.

@warp-core
Copy link
Contributor

I can implement this (though, first, I think I'd like to make a special object for licenses) but maybe not until after the next release (we have enough PRs open, I have enough PRs open) unless Zitchas has specific plans to use it in 0.9.15?

@Zitchas
Copy link
Member Author

Zitchas commented Aug 16, 2022

That sounds good. This isn't an immediate 0.9.15 thing, unless this release takes a lot longer. It is mainly a preparatory capability for when we get license restrictions on Korath outfits and ships. Which I don't think is going to be a .15 thing unless this release takes a lot longer than it currently looks.

@Quantumshark
Copy link
Collaborator

I'd like the ability to do this on a specific basis (as in, specifically by outfit rather than just by outfitter), and I'd also like it to be extended to shipyards as well as outfitters.
What I specifically want to be able to do is have one specific ship sold in two outfitters, but have it require different licenses in each.

@ravenshining
Copy link
Member

An alternative way of handling this could be with outfit variants, e.g. as in #5849 , which is another extremely useful potential feature. Along with the fix #9682 , and a way to remove attributes in variants, your outfitter could simply specify a variant with different licence requirements.

I asked about this issue on Discord and was told that handling licences in the outfitter rather than on a planet- or outfit- level would involve a lot more code? I honestly have no idea what that's about, but if it's handled with variants maybe that problem would be averted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A suggestion for new content or functionality that requires code changes
Projects
None yet
Development

No branches or pull requests

5 participants