FCTV Toggle inappropriate

Toggle inappropriate does not appear to work. Remote does not refresh, channels remain locked.

Possibly related, if you click on a channel you are already on, it goes to a random channel instead of the "next" part of the same channel. It also hides the remote when it does this.

changed the description

I thought the concubine was supposed to bring the remote? I do have a concubine.

My current idea is rather brutish but meets the requirements, currently in local testing.
@Pregmodder
The Within the slave shopping channel the premium virgin slave has their devotion defined twice, first 40,60 and then 60,90 for some reason.
Also a few slaves are missing setters that might be considered standard e.g _slave.origin = "You purchased $him from FCTV's Home Slave Shopping stream channel.".
_slave.race is complaining about being undefined for some reason (currently on MILF/show == 5), while also showing a daughter slave.

Take the first devotion. Second will really skew the price and is a bit unrealistic.

Add origins.

Not sure about the race issue. @lowercasedonkey, the vector art mock up maybe?

Good guess. Looking at it I think it's:

<<elseif _show <= 8>> /* huge balls */
	<<= addA(_slave.race)>>

Not all shows less than 8 actually generate _slave. The error isn't specific to race, it just seems like _slave isn't defined. I think channel 7 would throw this error.

edit: hmm, no that can't be it. It generates on less than 8 too.

edit2: _slaveOneTimeMaxAge, does this work as a local? Seems to be unique to this file.

@lowercasedonkey

_slaveOneTimeMaxAge, does this work as a local?

Not likely, I have it fixed locally though, along with a few other issues found along the way.

For clarity's sake, the issue displayed in the screenshot is while on my local improvement branch.

@Pregmodder
Another couple sources of confusion:
<<set $activeSlave.chem = 0>> is not set for "huge balls"/"discount young hottie"

<<set $activeSlave.health.condition = 75>>
		<<set $activeSlave.health.health = $activeSlave.health.condition - $activeSlave.health.shortDamage - $activeSlave.health.longDamage>>

is not set for "huge balls"/"discount young hottie"

Also <<set $activeSlave.career is the string "a slave" for "premium virgin", while in all other calls it is $activeSlave.career = setup.youngCareers.random(). "discount young hottie" and "huge balls" do not have it, unless 25 is somehow not young but 24 ("superfetation') is? "MILF" has it set as $activeSlave.career = setup.youngCareers.random() for some reason.

What is the point of <<set $activeSlaveOneTimeMinAge = ($fertilityAge + 3/2)>> when <<set $one_time_age_overrides_pedo_mode = 1>> and $activeSlaveOneTimeMaxAge >= 18?

chem will mostly be inherited as 0 from base slaveGen, I believe.

Health Should be set for those.

Not sure what to do with the careers. Maybe locking them all to "a slave" is the way to go.

It locks the range. Wouldn't advise messing with it.

Give 'a slave' careers to all but the disobedient bargain. I think the other slaves would have been slaves for a long time, especially the double virgin.

@Pregmodder
I am looking for some random function call related clarifications:
[0] Is random(0,x)+1 equivalent to random(1,x+1)?
[1] Is random(7) equivalent to random(0,7)?
Both though there are no show == 0 calls, show == 1 is the lowest.

Also <<set $oneTimeDisableDisability = 1>> is shared amoungst the five or so slave objects.

Yes to [0], maybe to [1]. I think it defaults to 0 in that case since .pluck(7) would grab something between 0 and 7 inclusive.

@svornost
Where would Math.clamp() be defined? src/js`dTree.min.js would appear to be the likely answer yet it any of the hits for "clamp" do not seem to use the same parameters (unless n equals e somehow)?

It's an extension method courtesy of Sugarcube. https://www.motoslave.net/sugarcube/2/docs/#methods-math-method-clamp

mentioned in issue #1589 (closed)

mentioned in merge request !6460 (merged)

mentioned in commit 16ff9967

closed via merge request !6460 (merged)