removed useless functions, ensured we use pointers on chaining functions

genericproblem.go

@@ -8,7 +8,7 @@ type GenericProblem struct {
 	Instance string `json:"instance,omitempty"` // A URI reference that identifies the specific ocurrence. Does not need to derefrence to something
 }
 
-func (problem GenericProblem) WithType(t string) GenericProblem {
+func (problem *GenericProblem) WithType(t string) *GenericProblem {
 	problem.Type = t
 	return problem
 }

validationproblem.go

@@ -20,7 +20,7 @@ type ValidationProblem struct {
 	ValidationErrors []ValidationError `json:"validationerrors,omitempty"` // a list of the validation errors that occurred
 }
 
-func (problem ValidationProblem) WithType(t string) ValidationProblem {
+func (problem *ValidationProblem) WithType(t string) *ValidationProblem {
 	problem.Type = t
 	return problem
 }
@@ -45,26 +45,6 @@ func (problem *ValidationProblem) WithInstance(instance string) *ValidationProbl
 	return problem
 }
 
-func (problem *ValidationProblem) ProblemType() string {
-	return problem.Type
-}
-
-func (problem *ValidationProblem) ProblemTitle() string {
-	return problem.Title
-}
-
-func (problem *ValidationProblem) ProblemStatus() int {
-	return problem.Status
-}
-
-func (problem *ValidationProblem) ProblemDetail() string {
-	return problem.Detail
-}
-
-func (problem *ValidationProblem) ProblemInstance() string {
-	return problem.Instance
-}
-
 func FromValidationErrors(validationErrors validator.ValidationErrors) ValidationProblem {
 	var errs []ValidationError