TeamUlysses / ulib

ULib: A Lua library for more rapid development on Garry's Mod servers
http://ulyssesmod.net
Other
105 stars 55 forks source link

Error #10

Closed deinemudda32 closed 6 years ago

deinemudda32 commented 8 years ago

[ERROR] lua/includes/extensions/math.lua:46: attempt to compare nil with number

  1. Clamp - lua/includes/extensions/math.lua:46
    1. ValueChanged - addons/ulx/lua/ulx/modules/cl/xgui_helpers.lua:261
    2. SetValue - addons/ulx/lua/ulx/modules/cl/xlib.lua:587
      1. unknown - addons/ulx/lua/ulx/modules/cl/xlib.lua
Nayruden commented 8 years ago

What were you doing when you received this error?

On Wed, Jan 27, 2016 at 12:57 PM, David Dorn notifications@github.com wrote:

[ERROR] lua/includes/extensions/mathlua:46: attempt to compare nil with number 1 Clamp - lua/includes/extensions/mathlua:46 2 ValueChanged - addons/ulx/lua/ulx/modules/cl/xgui_helperslua:261 3 SetValue - addons/ulx/lua/ulx/modules/cl/xliblua:587 4 unknown - addons/ulx/lua/ulx/modules/cl/xliblua

— Reply to this email directly or view it on GitHub https://github.com/TeamUlysses/ulib/issues/10.

deinemudda32 commented 8 years ago

Happend while browsing through the command categories randomly.

ghost commented 8 years ago

Is this fixed now got same issue

Nayruden commented 8 years ago

The issue shows as closed if it is fixed On Mon, Jul 4, 2016 at 18:25 Gamerpro2266 notifications@github.com wrote:

Is this fixed now got same issue

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/TeamUlysses/ulib/issues/10#issuecomment-230359937, or mute the thread https://github.com/notifications/unsubscribe/AABAz2HKbwFUaxktEhcA1ylxGddv98D1ks5qSYg8gaJpZM4HNpC0 .

Nayruden commented 7 years ago

@SticklyMan have you had a chance to look into this?

SticklyMan commented 6 years ago

I was only able to reproduce this after making a time-based NumArg restriction (ulx ban) with the same min and max that was not 0, then tried to set the time using the numberwang thing.

Not sure if that was the original issue, but the fix should work regardless. (Basically, there was invalid input sent to the code to enforce min/max constraints)