added validation; DB is global now
This commit is contained in:
@@ -13,18 +13,15 @@ import (
|
||||
)
|
||||
|
||||
type MembershipController struct {
|
||||
service services.MembershipService
|
||||
Service services.MembershipService
|
||||
}
|
||||
|
||||
type MembershipData struct {
|
||||
APIKey string `json:"api_key"`
|
||||
Model models.SubscriptionModel `json:"model"`
|
||||
}
|
||||
|
||||
func NewMembershipController(service services.MembershipService) *MembershipController {
|
||||
return &MembershipController{service}
|
||||
}
|
||||
|
||||
func (uc *MembershipController) RegisterSubscription(c *gin.Context) {
|
||||
func (mc *MembershipController) RegisterSubscription(c *gin.Context) {
|
||||
var regData MembershipData
|
||||
if err := c.ShouldBindJSON(®Data); err != nil {
|
||||
logger.Error.Printf("Couln't decode subscription data: %v", err)
|
||||
@@ -46,7 +43,7 @@ func (uc *MembershipController) RegisterSubscription(c *gin.Context) {
|
||||
logger.Info.Printf("registering subscription: %+v", regData)
|
||||
|
||||
// Register Subscription
|
||||
id, err := uc.service.RegisterSubscription(®Data.Model)
|
||||
id, err := mc.Service.RegisterSubscription(®Data.Model)
|
||||
if err != nil {
|
||||
logger.Error.Printf("Couldn't register Membershipmodel: %v", err)
|
||||
c.JSON(http.StatusInternalServerError, "Couldn't register Membershipmodel")
|
||||
|
||||
@@ -4,28 +4,23 @@ import (
|
||||
"GoMembership/internal/models"
|
||||
"GoMembership/internal/services"
|
||||
|
||||
"github.com/gin-gonic/gin"
|
||||
"net/http"
|
||||
|
||||
"github.com/gin-gonic/gin"
|
||||
|
||||
"GoMembership/pkg/logger"
|
||||
)
|
||||
|
||||
type UserController struct {
|
||||
service services.UserService
|
||||
emailService services.EmailService
|
||||
consentService services.ConsentService
|
||||
bankAccountService services.BankAccountService
|
||||
membershipService services.MembershipService
|
||||
Service services.UserServiceInterface
|
||||
EmailService *services.EmailService
|
||||
ConsentService services.ConsentServiceInterface
|
||||
BankAccountService services.BankAccountServiceInterface
|
||||
MembershipService services.MembershipServiceInterface
|
||||
}
|
||||
|
||||
type RegistrationData struct {
|
||||
User models.User `json:"user"`
|
||||
BankAccount models.BankAccount `json:"bank_account"`
|
||||
Membership models.Membership `json:"membership"`
|
||||
}
|
||||
|
||||
func NewUserController(service services.UserService, emailService *services.EmailService, consentService services.ConsentService, bankAccountService services.BankAccountService, membershipService services.MembershipService) *UserController {
|
||||
return &UserController{service, *emailService, consentService, bankAccountService, membershipService}
|
||||
User models.User `json:"user"`
|
||||
}
|
||||
|
||||
func (uc *UserController) RegisterUser(c *gin.Context) {
|
||||
@@ -37,10 +32,18 @@ func (uc *UserController) RegisterUser(c *gin.Context) {
|
||||
c.JSON(http.StatusBadRequest, gin.H{"error": "Couldn't decode userdata"})
|
||||
return
|
||||
}
|
||||
logger.Info.Printf("registering user: %v", regData.User)
|
||||
logger.Info.Printf("registering user: %#v", regData.User)
|
||||
|
||||
selectedModel, err := uc.MembershipService.GetModelByName(®Data.User.Membership.SubscriptionModel.Name)
|
||||
if err != nil {
|
||||
logger.Error.Printf("No subscription model found: %#v", err)
|
||||
c.JSON(http.StatusInternalServerError, gin.H{"error": "Not a valid subscription model"})
|
||||
return
|
||||
}
|
||||
|
||||
regData.User.Membership.SubscriptionModel = *selectedModel
|
||||
// Register User
|
||||
id, token, err := uc.service.RegisterUser(®Data.User)
|
||||
id, token, err := uc.Service.RegisterUser(®Data.User)
|
||||
if err != nil {
|
||||
logger.Error.Printf("Couldn't register User: %v", err)
|
||||
c.JSON(http.StatusInternalServerError, gin.H{"error": "Couldn't register User"})
|
||||
@@ -49,12 +52,12 @@ func (uc *UserController) RegisterUser(c *gin.Context) {
|
||||
regData.User.ID = id
|
||||
|
||||
// Register Bank Account
|
||||
_, err = uc.bankAccountService.RegisterBankAccount(®Data.BankAccount)
|
||||
/* _, err = uc.BankAccountService.RegisterBankAccount(®Data.User.BankAccount)
|
||||
if err != nil {
|
||||
logger.Error.Printf("Couldn't register bank account: %v", err)
|
||||
c.JSON(http.StatusInternalServerError, gin.H{"error": "Couldn't register User-BankAccount"})
|
||||
return
|
||||
}
|
||||
} */
|
||||
|
||||
// Register Consents
|
||||
var consents = [2]models.Consent{
|
||||
@@ -72,7 +75,7 @@ func (uc *UserController) RegisterUser(c *gin.Context) {
|
||||
},
|
||||
}
|
||||
for _, consent := range consents {
|
||||
_, err = uc.consentService.RegisterConsent(&consent)
|
||||
_, err = uc.ConsentService.RegisterConsent(&consent)
|
||||
if err != nil {
|
||||
logger.Error.Printf("Couldn't register consent: %v", err)
|
||||
c.JSON(http.StatusInternalServerError, gin.H{"error": "Couldn't register User-consent"})
|
||||
@@ -81,21 +84,21 @@ func (uc *UserController) RegisterUser(c *gin.Context) {
|
||||
}
|
||||
|
||||
// Register Membership
|
||||
_, err = uc.membershipService.RegisterMembership(®Data.Membership)
|
||||
/* _, err = uc.MembershipService.RegisterMembership(®Data.User.Membership)
|
||||
if err != nil {
|
||||
logger.Error.Printf("Couldn't register membership: %v", err)
|
||||
c.JSON(http.StatusInternalServerError, gin.H{"error": "Couldn't register User-membership"})
|
||||
return
|
||||
}
|
||||
} */
|
||||
|
||||
// Send notifications
|
||||
if err := uc.emailService.SendVerificationEmail(®Data.User, &token); err != nil {
|
||||
if err := uc.EmailService.SendVerificationEmail(®Data.User, &token); err != nil {
|
||||
logger.Error.Printf("Failed to send email verification email to user: %v", err)
|
||||
// Proceed without returning error since user registration is successful
|
||||
}
|
||||
|
||||
// Notify admin of new user registration
|
||||
if err := uc.emailService.NotifyAdminOfNewUser(®Data.User); err != nil {
|
||||
if err := uc.EmailService.NotifyAdminOfNewUser(®Data.User); err != nil {
|
||||
logger.Error.Printf("Failed to notify admin of new user registration: %v", err)
|
||||
// Proceed without returning error since user registration is successful
|
||||
}
|
||||
@@ -114,21 +117,21 @@ func (uc *UserController) VerifyMailHandler(c *gin.Context) {
|
||||
return
|
||||
}
|
||||
|
||||
user, err := uc.service.VerifyUser(&token)
|
||||
user, err := uc.Service.VerifyUser(&token)
|
||||
if err != nil {
|
||||
logger.Error.Printf("Cannot verify user: %v", err)
|
||||
c.JSON(http.StatusUnauthorized, gin.H{"error": "Cannot verify user"})
|
||||
return
|
||||
}
|
||||
|
||||
membership, err := uc.membershipService.FindMembershipByUserID(user.ID)
|
||||
membership, err := uc.MembershipService.FindMembershipByUserID(user.ID)
|
||||
if err != nil {
|
||||
logger.Error.Printf("Cannot get membership of user %v: %v", user.ID, err)
|
||||
c.JSON(http.StatusInternalServerError, gin.H{"error": "Cannot get Membership of user"})
|
||||
return
|
||||
}
|
||||
|
||||
uc.emailService.SendWelcomeEmail(user, membership)
|
||||
uc.EmailService.SendWelcomeEmail(user, membership)
|
||||
c.Status(http.StatusOK)
|
||||
|
||||
}
|
||||
@@ -138,7 +141,7 @@ func (uc *UserController) VerifyMailHandler(c *gin.Context) {
|
||||
Email string `json:"email"`
|
||||
Password string `json:"password"`
|
||||
}
|
||||
user, err := uc.service.AuthenticateUser(credentials.Email, credentials.Password)
|
||||
user, err := services.UserService.AuthenticateUser(credentials.Email, credentials.Password)
|
||||
if err != nil {
|
||||
http.Error(w, err.Error(), http.StatusUnauthorized)
|
||||
return
|
||||
@@ -153,7 +156,7 @@ func (uc *UserController) VerifyMailHandler(c *gin.Context) {
|
||||
http.Error(w, "Invalid user ID", http.StatusBadRequest)
|
||||
return
|
||||
}
|
||||
user, err := uc.service.GetUserByID(id)
|
||||
user, err := services.UserService.GetUserByID(id)
|
||||
if err != nil {
|
||||
http.Error(w, "User not found: "+err.Error(), http.StatusNotFound)
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user