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

Wearer setting safeword #986

Open
Medea-Destiny opened this issue Sep 1, 2023 · 4 comments
Open

Wearer setting safeword #986

Medea-Destiny opened this issue Sep 1, 2023 · 4 comments
Assignees
Labels
Discussion wanted Request for community feedback

Comments

@Medea-Destiny
Copy link
Collaborator

Medea-Destiny commented Sep 1, 2023

Currently owner auth is required to change safeword. I see no reason why the wearer should not be allowed to change their own safeword.

Code is found in oc_core here:

} else if(sChangetype == "safeword"){
            if(sChangevalue!=""){
                if(iNum == CMD_OWNER){
                    llMessageLinked(LINK_SET, LM_SETTING_SAVE, "global_safeword="+sChangevalue, "");
                    llMessageLinked(LINK_SET,NOTIFY,"1Safeword is now set to '"+sChangevalue,kID);

                    if(sChangevalue == "RED"){
                        llMessageLinked(LINK_SET, LM_SETTING_DELETE, "global_safeword","");
                    }

                    if(llToLower(sChangevalue) == "off"){
                        llMessageLinked(LINK_SET, LM_SETTING_SAVE, "global_safeworddisable=1", "");
                    } else {
                        llMessageLinked(LINK_SET, LM_SETTING_DELETE, "global_safeworddisable","");
                    }
                }
            } else {
                if(iNum == CMD_OWNER || kID == g_kWearer){
                    llMessageLinked(LINK_SET, NOTIFY, "0The safeword is current set to: '"+g_sSafeword+"'",kID);
                }
            }
        } 

If it's agreed we should let wearer change safeword, we should change if(iNum == CMD_OWNER){ to if(iNum==CMD_OWNER || kID==g_kWearer) and insert

if(iNum!=CMD_OWNER){
              llOwnerSay("No permission to switch off your own safeword, ask your owner!");
              return;
}

...prior to the if(llToLower(sChangevalue) == "off"){ line.

@Medea-Destiny Medea-Destiny self-assigned this Sep 1, 2023
@Medea-Destiny Medea-Destiny added Discussion wanted Request for community feedback 8.3 labels Sep 1, 2023
@Nix-Starlight
Copy link

Voted yes please

@Medea-Destiny
Copy link
Collaborator Author

@Nix-Starlight Well I know YOU want this, you're the person who suggested it! ;)

@NikkiLacrima
Copy link
Contributor

Since the wearer can use the safeword, there is NO reason why the wearer should not be allowed to change it. It is after all publicly displayed in Help/About.

Change approved.

@SilkieSabra
Copy link
Contributor

Since the owner can simply disable the safeword, I don't see any reason why an owned wearer shouldn't be able to change it. Let's move this to a pr please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion wanted Request for community feedback
Projects
None yet
Development

No branches or pull requests

4 participants