bot.*Context.GetRoles - Code Metrics - Inspection of "Added admin functions" - FlameInTheDark/dtbot - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d15815...179a5e )

by Viktor

created 2019-01-22 06:28 UTC

bot.*Context.GetRoles B

↳ Parent: bot/role.go

Complexity

Conditions

Size

Total Lines	14
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	7
eloc	10
nop	0
dl	0
loc	14
rs	8
c	0
b	0
f	0

package bot

import (
	"fmt"

	"github.com/bwmarrin/discordgo"
)

// UserRoles struct with array of user roles in guild
type UserRoles struct {
	Roles []*discordgo.Role
}

func (ctx *Context) IsAdmin() bool {

	if ctx.User.ID == ctx.Conf.General.AdminID {
		return true
	}
	return false
}

// GetRole returns UserRoles struct pointer

func (ctx *Context) GetRoles() *UserRoles {
	var userRoles = new(UserRoles)
	memb, err := ctx.Discord.GuildMember(ctx.Guild.ID, ctx.User.ID)
	if err != nil {
		fmt.Println("Getting member error: " + err.Error())
	}
	for _, grole := range ctx.Guild.Roles {
		for _, urole := range memb.Roles {
			if grole.ID == urole {
				userRoles.Roles = append(userRoles.Roles, grole)
			}
		}
	}
	return userRoles
}

// ExistsName checks if user role nema exists on user
func (r *UserRoles) ExistsName(name string) bool {
	for _, val := range r.Roles {
		if val.Name == name {
			return true
		}
	}
	return false
}


1			package bot
2
3			import (
4			"fmt"
5
6			"github.com/bwmarrin/discordgo"
7			)
8
9			// UserRoles struct with array of user roles in guild
10			type UserRoles struct {
11			Roles []*discordgo.Role
12			}
13
14			func (ctx *Context) IsAdmin() bool {
			0 ignored issues – show introduced 2019-01-22 06:30 UTC by Report Bug Copy Issue Report exported method Context.IsAdmin should have comment or be unexported Loading history...
15			if ctx.User.ID == ctx.Conf.General.AdminID {
16			return true
17			}
18			return false
19			}
20
21			// GetRole returns UserRoles struct pointer
			0 ignored issues – show introduced 2018-11-08 05:49 UTC by Report Bug Copy Issue Report comment on exported method Context.GetRoles should be of the form "GetRoles ..." Loading history...
22			func (ctx Context) GetRoles() UserRoles {
23			var userRoles = new(UserRoles)
24			memb, err := ctx.Discord.GuildMember(ctx.Guild.ID, ctx.User.ID)
25			if err != nil {
26			fmt.Println("Getting member error: " + err.Error())
27			}
28			for _, grole := range ctx.Guild.Roles {
29			for _, urole := range memb.Roles {
30			if grole.ID == urole {
31			userRoles.Roles = append(userRoles.Roles, grole)
32			}
33			}
34			}
35			return userRoles
36			}
37
38			// ExistsName checks if user role nema exists on user
39			func (r *UserRoles) ExistsName(name string) bool {
40			for _, val := range r.Roles {
41			if val.Name == name {
42			return true
43			}
44			}
45			return false
46			}
47

FlameInTheDark / dtbot

Push — master ( d15815...179a5e )

bot.*Context.GetRoles B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like