[ambn]

Originally Posted by plock I know there is a cvar to enable that so it remembers the choice, but as mentioned, some unwanted players may have the skin. So I think it would be easier and safer to add the a default skin when the flags are loaded, and let them choose later if they want another skin or disable the custom skin. As a VIP, it's kind of boring that everytime I enter the server, I have to choose the skin again, since I have only 1 options for each team. It's just user experience.

You can enable user saved skins so it saves their skin and set it back as they enter the server because the saved skin is preferred over the default skin

[plock]

Originally Posted by ambn You can enable user saved skins so it saves their skin and set it back as they enter the server because the saved skin is preferred over the default skin

I know that, but:

sm_map_dstay (Default "0") --> If Enabled, player's skin will stay on disconnect so they will still get their skin on reconnect (using this may cause unwanted skins on different people)

People with no flags may get a skin due to this problem, no?

[ambn]

Originally Posted by plock I know that, but: sm_map_dstay (Default "0") --> If Enabled, player's skin will stay on disconnect so they will still get their skin on reconnect (using this may cause unwanted skins on different people) People with no flags may get a skin due to this problem, no?

You may set sm_save_skins to 1 for that and about ur question its saved skin so if the user don't have access to select it then they can't save it.
Wait for the nexf release to get that permission problem fixed

[eryk172]

Can you add the use of random skins for players?

[ambn]

Originally Posted by eryk172 Can you add the use of random skins for players?

Yeah but how about permissions? Should they get a random one from all skins or just one from what they have access to?

[eryk172]

Players without random readiness random models, for example 4 selected ones. Players with permissions received skins in the file.

I had a few bugs during compilation, I corrected them. (SM 1.8 ) Maybe...

L 04/06/2018 - 10:43:12: Info (map "jail_xmf") (file "errors_20180406.log")
L 04/06/2018 - 10:43:12: [SM] Exception reported: invalid handle 0 (error: 4)
L 04/06/2018 - 10:43:12: [SM] Blaming: PlayerSkin.smx
L 04/06/2018 - 10:43:12: [SM] Call stack trace:
L 04/06/2018 - 10:43:12: [SM]   [0] ReadFileLine
L 04/06/2018 - 10:43:12: [SM]   [1] Line 1619, PlayerSkin.sp::AddFilesToDownload
L 04/06/2018 - 10:43:12: [SM]   [2] Line 297, PlayerSkin.sp::OnMapStart

[plock]

Originally Posted by eryk172 Players without random readiness random models, for example 4 selected ones. Players with permissions received skins in the file. I had a few bugs during compilation, I corrected them. (SM 1.8 ) Maybe... L 04/06/2018 - 10:43:12: Info (map "jail_xmf") (file "errors_20180406.log") L 04/06/2018 - 10:43:12: [SM] Exception reported: invalid handle 0 (error: 4) L 04/06/2018 - 10:43:12: [SM] Blaming: PlayerSkin.smx L 04/06/2018 - 10:43:12: [SM] Call stack trace: L 04/06/2018 - 10:43:12: [SM] [0] ReadFileLine L 04/06/2018 - 10:43:12: [SM] [1] Line 1619, PlayerSkin.sp::AddFilesToDownload L 04/06/2018 - 10:43:12: [SM] [2] Line 297, PlayerSkin.sp::OnMapStart

I will test it and tell if everything is working.

One thing that I missed was a generated cfg file to change the cvars. I read the code and I saw a lot of commented cvars, I didn't know they were disabled, because they are still in the post description and there was no cfg file generated.
I had to recompile it with the values I wanted.

Edit: For some reason it generated with the latest version, ignore.

I noticed a few functions are not being used, is that right?

PlayerSkin.sp(2367) : warning 203: symbol is never used: "GetUserCFlags"
PlayerSkin.sp(2367) : warning 203: symbol is never used: "IsFileMaterial"
PlayerSkin.sp(2367) : warning 203: symbol is never used: "IsFileModel"
PlayerSkin.sp(2367) : warning 203: symbol is never used: "IsValidFile"

[plock]

Found this issue:

L 04/06/2018 - 13:43:36: [SM] Exception reported: Client 2 is not in game
L 04/06/2018 - 13:43:36: [SM] Blaming: PlayerSkin.smx
L 04/06/2018 - 13:43:36: [SM] Call stack trace:
L 04/06/2018 - 13:43:36: [SM]   [0] GetClientTeam
L 04/06/2018 - 13:43:36: [SM]   [1] Line 630, plugin.sp::RoundStart

You should check if "IsClientInGame(client)" at line 627. It should fix the issue.

[eryk172]

I dit to lazy to delete variables.
I have sent this version because the original has such compiling errors:

// PlayerSkinON.sp(1108) : warning 213: tag mismatch
// PlayerSkinON.sp(1491) : warning 209: function "GiveBomb" should return a value
// PlayerSkinON.sp(2343) : warning 209: function "RemoveSavedSkins" should return a value
// PlayerSkinON.sp(2367) : warning 203: symbol is never used: "GetUserCFlags"
// PlayerSkinON.sp(2367) : warning 203: symbol is never used: "IsFileMaterial"
// Code size:            50112 bytes
// Data size:            59464 bytes
// Stack/heap size:      16384 bytes
// Total requirements:  125960 bytes
//
// 5 Warnings.
//
// Compilation Time: 0,39 sec
// ----------------------------------------

In the attachment I'm sending the version witch the fix from up post

[ambn]

UPDATED! Version 4.6.0 fix8** 4/7/2018
- Fixed an issue with user's permission being ignored and everyone was able to choose all skins from the menu.
- Fixed an issue with saved skins which it's arms would not set.
Note: delete automatic config generated for the plugin and let the plugin re-create it again.