feat: Lots of new useful features and fix by PUDGE133 · Pull Request #864 · ExMod-Team/EXILED

PUDGE133 · 2026-08-26T20:22:19Z

Description

Describe the changes

Several very useful extension methods.
Additional overloads of the Get method for Door.
Improving the Door.Get(GameObject) method
Implementation of new properties for Firearm
Created stubs for the Log class methods
Added several new teleportation methods for the Player
Implemented a new property Server.ReservedSlots
And also a few minor corrections in the code and in the documentation for methods and properties

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)
Documentations

Submission checklist

I have checked the project can be compiled
I have tested my changes and it worked as expected

Other

Still requires more testing

PUDGE133 · 2026-08-26T20:24:54Z

If you have any questions, you can write here or to me on Discord: scaletta2019

louis1706 · 2026-08-31T18:38:56Z

+        /// <summary>
+        /// Gets or sets a value indicating whether the magazine is attached from the weapon.
+        /// </summary>
+        public bool IsMagazineDeattached
+        {
+            get => !IsMagazineAttached;
+            set => IsMagazineAttached = !value;
+        }


This is useless

This is useless

If you think this section of code is useless, then let's think this code is useless too:

If you think this section of code is useless, then let's think this code is useless too:

It may be useless, but it is not beneficial to remove it because that would be a breaking change. There is also no reason to add new junk code.

If you think this section of code is useless, then let's think this code is useless too:

It may be useless, but it is not beneficial to remove it because that would be a breaking change. There is also no reason to add new junk code.

No, no, no... you've had enough of this "critical code." You shouldn't have added it and then been afraid to remove it. As long as these and similar properties exist in the code, the code I've submitted will comply with Exiled standards.

This is excellent code; it reduces the cognitive load when reading it in plugins. There is no problem with it.

louis1706 · 2026-08-31T18:40:44Z


-        /// <inheritdoc cref="Player.Count"/>
-        public static int PlayerCount => Player.Count;
+        /// <summary>
+        /// Gets the number of players excluding NPCs.
+        /// </summary>
+        public static int PlayerCount => Player.Dictionary.Values.Count(x => !x.IsNPC);


No please don't break all plugin that would use that

No please don't break all plugin that would use that

Okay, I'll change it back. However, I don't understand why this property originally included NPCs. It once took me quite a while to figure out what was causing the bug. The problem was that someone misnamed the property. You should have named it something that would make it clear it included NPCs as well. If necessary, I can change the documentation for this property.

louis1706 · 2026-08-31T18:41:25Z

+        {
+            if (gameObject != null)
+            {
+                // ParentDoor requires enabling "unsafe code"


Weird comment?

Weird comment?

I will remove this comment.

louis1706 · 2026-08-31T18:46:49Z

+            if (room.Type == RoomType.Surface)
+                return relativePos;


Surface is a functional room technically i only see the purpose of this for Unknown Room

Or if you do it with the surface let's do it to all non dynamic rooms (i prefer to only do it on null that would be unknown)

Surface and pocket

Surface is a functional room technically i only see the purpose of this for Unknown Room

Or if you do it with the surface let's do it to all non dynamic rooms (i prefer to only do it on null that would be unknown)

Surface and pocket

I don't understand what you didn't like about this code.

louis1706 · 2026-08-31T18:47:38Z

+        public static string ToHumanReadable(this TimeSpan timeSpan)
+        {
+            if (timeSpan.TotalHours < 1)
+                return timeSpan.ToString(@"mm\:ss");
+
+            if (timeSpan.TotalDays < 1)
+                return timeSpan.ToString(@"hh\:mm\:ss");
+
+            string daysPart = timeSpan.Days == 1 ? "1 day" : $"{timeSpan.Days} days";
+            string timePart = timeSpan.ToString(@"hh\:mm\:ss");
+            return $"{daysPart}, {timePart}";
+        }


How does it handle differences like language/ 24h or 12h Am/Pm

How does it handle differences like language/ 24h or 12h Am/Pm

Normal people use the 24-hour clock. In any case, I didn't intend for this method to be universal. If someone needs it, they can always add their own extension method to this class with a different name or a different overload. I don't see any problem with that.

Normal people use the 24-hour clock. In any case, I didn't intend for this method to be universal. If someone needs it, they can always add their own extension method to this class with a different name or a different overload. I don't see any problem with that.

It's an API meant for many people to use. A lot of this code is pretty pointless or not helpful to many people.

Normal people use the 24-hour clock. In any case, I didn't intend for this method to be universal. If someone needs it, they can always add their own extension method to this class with a different name or a different overload. I don't see any problem with that.

It's an API meant for many people to use. A lot of this code is pretty pointless or not helpful to many people.

Write your own method and add it. Be sure to take into account the 250 official languages. Keep in mind that each language has its own vocabulary and grammar. I used one specific method for 99% of tasks. But you can add your own.

louis1706 · 2026-08-31T18:48:13Z

+        public static bool NextBool(this Random rnd)
+        {
+            return rnd.Next(2) == 0;
+        }


Pretty sure this is very badly optimised

Pretty sure this is very badly optimised

I don't see any particular problem here, but you can do it like this:
(rnd.Next() & 1) == 0

louis1706 · 2026-08-31T18:48:49Z

+        /// <summary>
+        /// Generate a random float.
+        /// </summary>
+        /// <param name="rnd"><see cref="Random"/> object.</param>
+        /// <param name="min">Minimum value.</param>
+        /// <param name="max">Maximum value.</param>
+        /// <returns>Random value between minimum and maximum.</returns>
+        public static float NextFloat(this Random rnd, float min, float max)
+        {
+            return (float)((rnd.NextDouble() * (max - min)) + min);
+        }
+
+        /// <summary>
+        /// Generate a random float.
+        /// </summary>
+        /// <param name="rnd"><see cref="Random"/> object.</param>
+        /// <param name="min">Minimum value.</param>
+        /// <param name="max">Maximum value.</param>
+        /// <returns>Random value between minimum and maximum.</returns>
+        public static float NextFloat(this Random rnd, double min, float max)
+        {
+            return (float)((rnd.NextDouble() * (max - min)) + min);
+        }
+
+        /// <summary>
+        /// Generate a random float.
+        /// </summary>
+        /// <param name="rnd"><see cref="Random"/> object.</param>
+        /// <param name="min">Minimum value.</param>
+        /// <param name="max">Maximum value.</param>
+        /// <returns>Random value between minimum and maximum.</returns>
+        public static float NextFloat(this Random rnd, float min, double max)
+        {
+            return (float)((rnd.NextDouble() * (max - min)) + min);
+        }
+
+        /// <summary>
+        /// Generate a random float.
+        /// </summary>
+        /// <param name="rnd"><see cref="Random"/> object.</param>
+        /// <param name="min">Minimum value.</param>
+        /// <param name="max">Maximum value.</param>
+        /// <returns>Random value between minimum and maximum.</returns>
+        public static float NextFloat(this Random rnd, double min, double max)
+        {
+            return (float)((rnd.NextDouble() * (max - min)) + min);
+        }


Doesn't Unity Random already handle some of these ?

Doesn't Unity Random already handle some of these ?

Because not everyone likes Random from unity.

Because not everyone likes Random from unity.

??? What is wrong with unity randoms?

Because not everyone likes Random from unity.

??? What is wrong with unity randoms?

What difference does it make to you? What changes for you? Use the randomness you want.

louis1706 · 2026-08-31T18:49:29Z

+        /// <summary>
+        /// Perform an action on each element of a collection.
+        /// </summary>
+        /// <typeparam name="T">Type of <see cref="IEnumerable{T}"/> elements.</typeparam>
+        /// <param name="enumerable"><see cref="IEnumerable{T}"/> in this collection, the elements will perform actions.</param>
+        /// <param name="action">Action that needs to be performed.</param>
+        public static void ForEach<T>(this IEnumerable<T> enumerable, Action<T> action)
+        {
+            if (enumerable is null || action is null)
+                return;
+
+            foreach (T e in enumerable)
+                action(e);
+        }


Isn't that already a things ?

Isn't that already a things ?

This method is defined only for arrays and sheets. I've made it available to all IEnumerable<T> collections.

Mrhootyhoot1 · 2026-09-06T02:42:01Z

            where T : Door => Get(doorVariant) as T;

+        /// <summary>
+        /// Gets the door object associated with a specific <see cref="ButtonVariant"/>, or creates a new one if there isn't one.


Why would you create a new door if there is not a door associated with that buttonvariant?

Why would you create a new door if there is not a door associated with that buttonvariant?

fixed

Mrhootyhoot1 · 2026-09-06T02:43:19Z

        /// </summary>
        /// <param name="message">The message to be sent.</param>
-        public static void Error(string message) => Send($"[{Assembly.GetCallingAssembly().GetName().Name}] {message}", Discord.LogLevel.Error, ConsoleColor.DarkRed);
+        public static void Error(string message = "") => Send($"[{Assembly.GetCallingAssembly().GetName().Name}] {message}", Discord.LogLevel.Error, ConsoleColor.DarkRed);


Why would anyone call a method meant to print to the console without passing a string to the method to be printed?

Why would anyone call a method meant to print to the console without passing a string to the method to be printed?

Have you ever split console logs to make them easier to read?

github-actions Bot added API CustomModules labels Aug 26, 2026

github-actions Bot assigned PUDGE133 Aug 26, 2026

louis1706 reviewed Aug 31, 2026

View reviewed changes

several corrections in the code

de125a4

PUDGE133 force-pushed the NewFeatures branch from 7d85cb3 to de125a4 Compare September 1, 2026 09:43

Mrhootyhoot1 reviewed Sep 6, 2026

View reviewed changes

PUDGE133 added 2 commits September 6, 2026 17:46

fix summary door

de9c314

flags attribute on firearmtype

2f75e38

Uh oh!

Conversation

PUDGE133 commented Aug 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Types of changes

Submission checklist

Other

Uh oh!

PUDGE133 commented Aug 26, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Mrhootyhoot1 Sep 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

PUDGE133 Sep 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

PUDGE133 Sep 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

PUDGE133 Sep 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

PUDGE133 commented Aug 26, 2026 •

edited

Loading

Mrhootyhoot1 Sep 6, 2026 •

edited

Loading

PUDGE133 Sep 6, 2026 •

edited

Loading

PUDGE133 Sep 1, 2026 •

edited

Loading

PUDGE133 Sep 1, 2026 •

edited

Loading