Jump to content

Recommended Posts

Posted (edited)

I'd like to suggest the addition of a new permission to disable monster drops for that group.

bonus permissions xD \/

  • Cannot pickup items
  • Cannot buy/sell items at npc (opening the trade windows is okay tho)
  • Do not gain experience
  • Ignore logout delay

Edited by EvilPuncker
Posted (edited)

Hi, this diff adds 3 new pemissions

disable_pickup: true
disable_exp: true
disable_store: true

I've tried doing the logout delay part, but the client thinks that the character is still logged in (its either too fast or i'm missing something)

thanks alot! btw I've just noticed that the logout delay bypass is kinda useless :P so I'll remove it from the list, now just missing the permission to disable monster drops

Edited by EvilPuncker
Posted

Disabling monster drops would be useless, it can be bypassed by doing the following:

1. GM attacks the monsters until it is about to die.

2. A normal player will be the one to last hit the monster.

3. Viola, bypassed the restriction.

It is better to disable pvm all the way, thus using 'disable_pvm'.

Posted

Disabling monster drops would be useless, it can be bypassed by doing the following:

1. GM attacks the monsters until it is about to die.

2. A normal player will be the one to last hit the monster.

3. Viola, bypassed the restriction.

It is better to disable pvm all the way, thus using 'disable_pvm'.

Well, I was thinking about monsters killed by GM summons/homunculus, right click on monster etc

suggesting to disable gm buffing.

eg.

a gm will buff a normal character

I think its better to have something like "can_use_skills_on_players" xD

  • Upvote 1
Posted

Yes, I've updated my copy to be able to use the following settings

 disable_pickup: true
 disable_exp: true
 disable_store: true
 disable_skill_usage: true
 disable_drops: true

If you'd like more, please reply with what you'd like to be added. That way, I don't have to upload a new diff everytime a new option is added.

  • Upvote 1
Posted

Yes, I've updated my copy to be able to use the following settings

 disable_pickup: true
 disable_exp: true
 disable_store: true
 disable_skill_usage: true
 disable_drops: true

If you'd like more, please reply with what you'd like to be added. That way, I don't have to upload a new diff everytime a new option is added.

that is cool already, just need any admin to approve this idea :D

Posted

Yes, I've updated my copy to be able to use the following settings

 disable_pickup: true
 disable_exp: true
 disable_store: true
 disable_skill_usage: true
 disable_drops: true

If you'd like more, please reply with what you'd like to be added. That way, I don't have to upload a new diff everytime a new option is added.

when will you upload the new diff? :D

  • 1 month later...
Posted

Disabling monster drops would be useless, it can be bypassed by doing the following:

1. GM attacks the monsters until it is about to die.

2. A normal player will be the one to last hit the monster.

3. Viola, bypassed the restriction.

It is better to disable pvm all the way, thus using 'disable_pvm'.

Well, I was thinking about monsters killed by GM summons/homunculus, right click on monster etc

suggesting to disable gm buffing.

eg.

a gm will buff a normal character

I think its better to have something like "can_use_skills_on_players" xD

+1 :) other GM used "Iduns of apple" to player, Player got to high HP and use Gate of Hell and other skills (abuse) :(

Posted

Would using @mail be covered under the can_trade permission? If not, then a permission to disable mail would be nice.

Thinking of some new system that can be made for RO that utilizes these permissions and not being to use the mail would be an awesome addition.

And clydelion, why not have these permissions added in official to the SVN? I don't see the harm in adding more settings to the permission systems. I can actually add your work in myself, if you can give me permission to do.

Posted (edited)

sorry a bit off topic

how about making this setting :

exp_multiplier : 1000

will multiply all exp earning (mob,quest) 10*current rate

Edited by serakh00
Posted

Would using @mail be covered under the can_trade permission? If not, then a permission to disable mail would be nice.

Yes it is covered. I also propose changing it to can_give instead of can_trade.

And clydelion, why not have these permissions added in official to the SVN? I don't see the harm in adding more settings to the permission systems. I can actually add your work in myself, if you can give me permission to do.

Yes, please. :D

Posted

Some small notes on the patch to consider before it is commited:

Naming like: pc_has_permission(sd,PC_PERM_DISABLE_STORE), is a bit confusing I think. So, has permission to not to have permission to open store, or something like that. I think it is better to avoid unnecessary negations. In my opinion permissions like: pickup, exp, store etc are more expressive than the disable_ prefixed counterparts.

The default settings should almost always preserve the original behavior. For instance, if gm's were able to pick up items until now, the default for disable_pickup should be false in groups.conf.

And finaly, if you increased the number of digits of an enum, append the additional 0's to the beginning of the other constants to match the style.

Other than that, it looks ok for me, nice work :)

Edit: just noticed, there are already some other disable_ prefixed permissions, however I still do not think, those are the right way to handle them. Alway better to keep it simple.

Posted

Some small notes on the patch to consider before it is commited:

Naming like: pc_has_permission(sd,PC_PERM_DISABLE_STORE), is a bit confusing I think. So, has permission to not to have permission to open store, or something like that. I think it is better to avoid unnecessary negations. In my opinion permissions like: pickup, exp, store etc are more expressive than the disable_ prefixed counterparts.

The default settings should almost always preserve the original behavior. For instance, if gm's were able to pick up items until now, the default for disable_pickup should be false in groups.conf.

And finaly, if you increased the number of digits of an enum, append the additional 0's to the beginning of the other constants to match the style.

Other than that, it looks ok for me, nice work :)

Edit: just noticed, there are already some other disable_ prefixed permissions, however I still do not think, those are the right way to handle them. Alway better to keep it simple.

I checked with Ind on this but turns out I not allowed to commit them cause my SVN access is only for documentation changes. =/

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...