Change Details

Expected: /faction_mod_member Wolfe 4 /faction_list_members 10035 RETURN: [SERVER, [ADMIN COMMAND] [SUCCESS] Libretarian Alliance: {EpicBeard=>FactionPermission [playerUID=EpicBeard, roleID=1], Jtkenney_1=>FactionPermission [playerUID=Jtkenney_1, roleID=1], SnowMaster=>FactionPermission [playerUID=SnowMaster, roleID=1], Wolfe=>FactionPermission [playerUID=Wolfe, roleID=4], zachary=>FactionPermission [playerUID=zachary, roleID=2], zack=>FactionPermission [playerUID=zack, roleID=0]}, 0] Actual: /faction_mod_member Wolfe 4 /faction_list_members 10035 RETURN: [SERVER, [ADMIN COMMAND] [SUCCESS] Libretarian Alliance: {EpicBeard=>FactionPermission [playerUID=EpicBeard, roleID=1], Jtkenney_1=>FactionPermission [playerUID=Jtkenney_1, roleID=1], SnowMaster=>FactionPermission [playerUID=SnowMaster, roleID=1], Wolfe=>FactionPermission [playerUID=Wolfe, roleID=3], zachary=>FactionPermission [playerUID=zachary, roleID=2], zack=>FactionPermission [playerUID=zack, roleID=0]}, 0] Rationale: A faction's roleID should match integer used in mod_member command. To behave as-if 0=1, 1=2, 2=3, 3=4, and 4=5 is not only an expected behavior, it's just downright confusing. Could this also be contributing to T149 ?

**Expected**: /faction_mod_member Wolfe 4 /faction_list_members 10035 RETURN: [SERVER, [ADMIN COMMAND] [SUCCESS] Libretarian Alliance: {EpicBeard=>FactionPermission [playerUID=EpicBeard, roleID=1], Jtkenney_1=>FactionPermission [playerUID=Jtkenney_1, roleID=1], SnowMaster=>FactionPermission [playerUID=SnowMaster, roleID=1], Wolfe=>FactionPermission [playerUID=Wolfe, roleID=4], zachary=>FactionPermission [playerUID=zachary, roleID=2], zack=>FactionPermission [playerUID=zack, roleID=0]}, 0] **Actual**: /faction_mod_member Wolfe 4 /faction_list_members 10035 RETURN: [SERVER, [ADMIN COMMAND] [SUCCESS] Libretarian Alliance: {EpicBeard=>FactionPermission [playerUID=EpicBeard, roleID=1], Jtkenney_1=>FactionPermission [playerUID=Jtkenney_1, roleID=1], SnowMaster=>FactionPermission [playerUID=SnowMaster, roleID=1], Wolfe=>FactionPermission [playerUID=Wolfe, roleID=3], zachary=>FactionPermission [playerUID=zachary, roleID=2], zack=>FactionPermission [playerUID=zack, roleID=0]}, 0] **Rationale**: A faction's roleID should match integer used in mod_member command. To behave as-if 0=1, 1=2, 2=3, 3=4, and 4=5 is not only an expected behavior, it's just downright confusing. **Could this also be contributing to** T149 ?

**Expected:**: /faction_mod_member Wolfe 4 /faction_list_members 10035 RETURN: [SERVER, [ADMIN COMMAND] [SUCCESS] Libretarian Alliance: {EpicBeard=>FactionPermission [playerUID=EpicBeard, roleID=1], Jtkenney_1=>FactionPermission [playerUID=Jtkenney_1, roleID=1], SnowMaster=>FactionPermission [playerUID=SnowMaster, roleID=1], Wolfe=>FactionPermission [playerUID=Wolfe, roleID=4], zachary=>FactionPermission [playerUID=zachary, roleID=2], zack=>FactionPermission [playerUID=zack, roleID=0]}, 0] **Actual:**: /faction_mod_member Wolfe 4 /faction_list_members 10035 RETURN: [SERVER, [ADMIN COMMAND] [SUCCESS] Libretarian Alliance: {EpicBeard=>FactionPermission [playerUID=EpicBeard, roleID=1], Jtkenney_1=>FactionPermission [playerUID=Jtkenney_1, roleID=1], SnowMaster=>FactionPermission [playerUID=SnowMaster, roleID=1], Wolfe=>FactionPermission [playerUID=Wolfe, roleID=3], zachary=>FactionPermission [playerUID=zachary, roleID=2], zack=>FactionPermission [playerUID=zack, roleID=0]}, 0] **Rationale:**: A faction's roleID should match integer used in mod_member command. To behave as-if 0=1, 1=2, 2=3, 3=4, and 4=5 is not only an expected behavior, it's just downright confusing. **Could this also be contributing to** T149 ?