Skip to content
This repository has been archived by the owner on Dec 19, 2022. It is now read-only.

ckSurf overrides ALL Admin-Tag plugins #14

Closed
Wh1t3Rose opened this issue Oct 7, 2017 · 25 comments
Closed

ckSurf overrides ALL Admin-Tag plugins #14

Wh1t3Rose opened this issue Oct 7, 2017 · 25 comments

Comments

@Wh1t3Rose
Copy link

Hello,

First off I know this isn't your fault. This has been an issue since the original ckSurf plugin. But can you either completely remove the Admin tags function or change they way it handles tags?

Even when I set AdminTags to 0 in the cfg file, this plugin break compatibility with every AdminTags plugin out there that modifies tags in the scoreboard. I have tried even some very recent plugins on alliedmodders like these:

https://forums.alliedmods.net/showpost.php?p=2421587&postcount=61
https://forums.alliedmods.net/showthread.php?t=293563
https://forums.alliedmods.net/showthread.php?t=241345&page=5

All of these get overridden by ckSurf. You might see your custom tag from the above plugins for a like a split second when loading into the map but then ckSurf takes over.

Thanks!

@tomtko
Copy link
Owner

tomtko commented Oct 7, 2017

This isn't just admin tags, it's because of that chat processing, however with this plugin it's possible for admins to set their "custom title" to "Admin" which will pretty much do the same thing as what you're asking. sm_customtitle "my cool title"

@Wh1t3Rose
Copy link
Author

Wh1t3Rose commented Oct 7, 2017

Yes I am aware of that feature. But I like using a proper admin tag plugin because it's automatic. It's based of flags and steam IDs and it forces my admins to wear their admin tags. Hence they can't unequip it.

It's a very bad practice to break other plugins for your own system. (Again I know this isn't your fault. It's been around since the start).

@tomtko
Copy link
Owner

tomtko commented Oct 8, 2017

I can add this feature if you like, but changing the chat processing is a bit of a task considering how much the original ckSurf relies on it.

@Wh1t3Rose
Copy link
Author

Wh1t3Rose commented Oct 9, 2017

@fluffyst See the issue I made for Nikoo's plugin here. Seems someone has a fix!

@tomtko
Copy link
Owner

tomtko commented Oct 9, 2017

@sandman007 check my reply in that, does that solution suit your needs? I could also add a convar like ck_enforce_default_tags which wouldn't allow people to change it from their default tag, if that is desired

@Wh1t3Rose
Copy link
Author

@fluffust That solution would be great! I have a confession. I'm actually not using your timer currently. I'm using Nikoo's. But I was and still am planning to move to yours.

As for the convar. I think it would be a nice addition. I Probably won't use it since I know some of my Admins prefer to show off their rank.

Maybe if we can have that convar force the tag in the scoreboard but not in chat (so show player surf rank in chat) that would be nice.

But you decide what you want.

tomtko added a commit that referenced this issue Oct 9, 2017
@tomtko
Copy link
Owner

tomtko commented Oct 9, 2017

If you wouldn't mind, I tried my own implementation of this feature, 62e7186

https://github.com/fluffyst/Surftimer/tree/dev

Would help a lot if you tried it / bug tested it

@Wh1t3Rose
Copy link
Author

@fluffyst Sure thing! I'll give it a test and let you know! It's 2am here and I have class/work in 6 hours so I'll have to do it tomorrow night. But I will def get back to you.

Should I setup a fresh DB since I am using nikoo's plugin? I read somewhere you stated that a fresh DB is required but I figured that was if you were coming from 1.18?

@tomtko
Copy link
Owner

tomtko commented Oct 10, 2017

You can use a existing database, just have to alter some tables, maybe i'll add a way to upgrade from niko to mine

Edit: If you could, would you mind sending me your current database so I have some sample data to work with? Need to make sure existing times player times and stuff carry over.

@Wh1t3Rose
Copy link
Author

@fluffyst Sure here you go!

cksurf.zip

@tomtko
Copy link
Owner

tomtko commented Oct 10, 2017

Thanks! Is this the latest version, v1.20?

@Wh1t3Rose
Copy link
Author

@fluffyst yes it is.

@tomtko
Copy link
Owner

tomtko commented Oct 10, 2017

Alright cool, wanted to make sure before I committed, 48e9643

@Wh1t3Rose
Copy link
Author

@fluffyst Excellent! I read the commit. I will run the sql file and try out your plugin in a few hours.

@Wh1t3Rose
Copy link
Author

Wh1t3Rose commented Oct 18, 2017

Sorry I have been very busy the past week. I just performed the upgrade and while I couldn't do the testing myself today, I had an Admin confirm that the Timer is working as it should. I will do testing myself tomorrow.

However my Admin told me that tags weren't working. I suspect since they are "default" tags they should just get them automatically if they have the appropriate flag? I am using Sbans to manage my Groups etc

"Default Titles"
{
  "0"
  {
    "flag" "z"
    "title" "{default}[{darkred}Owner{default}]"
  }
  "1"
  {
    "flag" "m"
    "title" "{default}[{green}Sr-Admin{default}]"
  }
  "2"
  {
    "flag" "f"
    "title" "{default}[{green}Admin{default}]"
  }
  "3"
  {
    "flag" "b"
    "title" "{default}[{green}Mod{default}]"
  }
  "4"
  {
    "flag" "a"
    "title" "{default}[{green}VIP{default}]"
  }
}

@tomtko
Copy link
Owner

tomtko commented Oct 18, 2017

Is the ConVar ck_enforce_default_titles set to more than 0? it's 0 (disabled) by default

@Wh1t3Rose
Copy link
Author

Yes it is. I just logged onto my server to check. First time I joined it just show "unranked" next to my name in the scoreboard. I rejoined it and it just says my name in the scoreboard and no tag.

@tomtko
Copy link
Owner

tomtko commented Oct 19, 2017

Are you running the dev branch? Any errors?

@Wh1t3Rose
Copy link
Author

Wh1t3Rose commented Oct 19, 2017

Yep Dev branch. No errors that seemed related in the logs.

Edit: Only thing I could find that seemed relevant.

L 10/19/2017 - 03:01:55: [surftimer] Starting to load server settings
L 10/19/2017 - 03:01:58: [Surftimer] Finished loading server settings in 1.828125s
L 10/19/2017 - 03:09:27: [surftimer] Starting to load server settings
L 10/19/2017 - 03:09:30: [Surftimer] Finished loading server settings in 2.359375s
L 10/19/2017 - 03:12:05: [Surftimer] Loading STEAM_1:1:63647066 - Fracture settings
L 10/19/2017 - 03:12:05: [Surftimer] STEAM_1:1:63647066: Finished db_viewPersonalRecords in 0.265625s
L 10/19/2017 - 03:12:05: [Surftimer] STEAM_1:1:63647066: Finished db_viewPersonalBonusRecords in 0.015625s
L 10/19/2017 - 03:12:05: [Surftimer] STEAM_1:1:63647066: Skipping db_viewPersonalStageRecords (linear map)
L 10/19/2017 - 03:12:05: [Surftimer] STEAM_1:1:63647066: Finished db_viewPlayerPoints in 0.031250s
L 10/19/2017 - 03:12:05: [Surftimer] STEAM_1:1:63647066: Finished db_GetPlayerRank in 0.046875s
L 10/19/2017 - 03:12:05: [Surftimer] STEAM_1:1:63647066: Finished db_viewPlayerOptions in 0.015625s
L 10/19/2017 - 03:12:05: [Surftimer] STEAM_1:1:63647066: Finished db_CheckVIPAdmin in 0.046875s
L 10/19/2017 - 03:12:05: [Surftimer] STEAM_1:1:63647066: Finished db_viewCustomTitles in 0.015625s
L 10/19/2017 - 03:12:05: [Surftimer] STEAM_1:1:63647066: Finished db_viewCheckpoints in 0.031250s
L 10/19/2017 - 03:12:05: [Surftimer] Finished loading STEAM_1:1:63647066 - Fracture settings in 0.468750s

@tomtko
Copy link
Owner

tomtko commented Oct 19, 2017

The latest smx? I tested it before and worked fine for me, make sure ck_enforce_default_titles isn't set to above 3 as well

@Wh1t3Rose
Copy link
Author

I think I am using the latest smx but I will double check. I figured ck_enforce_default_titles would be a 0 or 1 value only?

@tomtko
Copy link
Owner

tomtko commented Oct 19, 2017

has 4 values, 0 is disabled, 1 is only use the titles in the chat, 2 is only use the titles in the scoreboard, 3 is use the titles in both chat and scoreboard

Edit: thinking about it though, maybe I should add it as a option in the default_titles.txt instead, "type" or something, i'm going to add this, wait a few mins

@Wh1t3Rose
Copy link
Author

Oh!!!! I am so stupid xD. So it was working but I had it set to show in chat only! I need to learn to read! The explanation was right there in the cfg file lol.

I gotta say I am super pleased by this plugin already! Just the built in tags have made me super happy. Thanks @fluffyst! I guess I can close this now. You now have a new fan!

@tomtko
Copy link
Owner

tomtko commented Oct 19, 2017

@sandman007 I have changed the functionality in the latest commit, ck_enforce_default_titles is now 0/1 (disable/enable) and you can now change individual titles display in default_titles.txt with

"type" "scoreboard"
"type" "chat"
"type" "both"

@Wh1t3Rose
Copy link
Author

Got It! I'll grab the update tomorrow. Bed time for me now. :P

olokos pushed a commit to olokos/Surftimer that referenced this issue Apr 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants