up

pkg/auth/portal.go

@@ -2,7 +2,6 @@ package auth
 
 import (
 	"errors"
-	"log"
 
 	"github.com/gofiber/fiber/v2"
 	"github.com/sujit-baniya/fiber-boilerplate/app"
@@ -15,20 +14,11 @@ const portalSessionKey = "portal_user_id"
 func PortalUser(c *fiber.Ctx) (*models.PortalUser, error) {
 	store := app.Http.Session.Get(c)
 	raw := store.Get(portalSessionKey)
-	log.Printf("[PORTAL SESSION] GET %s → cookie=%q raw=%v", c.Path(), c.Cookies("session_id"), raw)
 	if raw == nil {
 		return nil, errors.New("no portal session")
 	}
-	var id uint
-	switch v := raw.(type) {
-	case uint:
-		id = v
-	case int64:
-		id = uint(v)
-	case float64:
-		id = uint(v)
-	default:
-		log.Printf("[PORTAL SESSION] type assertion failed: %T", raw)
+	id, ok := raw.(uint)
+	if !ok {
 		return nil, errors.New("invalid portal session")
 	}
 	u, err := models.GetPortalUserByID(id)
@@ -45,18 +35,8 @@ func PortalUser(c *fiber.Ctx) (*models.PortalUser, error) {
 // SetPortalSession guarda el portal_user_id en la sesión.
 func SetPortalSession(c *fiber.Ctx, userID uint) error {
 	store := app.Http.Session.Get(c)
-	// Regenerar ID de sesión para evitar session fixation
-	if err := store.Regenerate(); err != nil {
-		log.Printf("[PORTAL SESSION] Regenerate error: %v", err)
-		return err
-	}
 	store.Set(portalSessionKey, userID)
-	if err := store.Save(); err != nil {
-		log.Printf("[PORTAL SESSION] Save error: %v", err)
-		return err
-	}
-	log.Printf("[PORTAL SESSION] SET userID=%d cookie=%q", userID, c.Cookies("session_id"))
-	return nil
+	return store.Save()
 }
 
 // DestroyPortalSession elimina la sesión del portal.

pkg/models/portal_user.go

@@ -14,20 +14,15 @@ import (
 // - Rol "partner": accede a los proyectos de todos sus ClienteIDs via PortalAccesos.
 type PortalUser struct {
 	gorm.Model
-	Nombre        string         `json:"nombre"        gorm:"column:nombre;not null"`
-	Email         string         `json:"email"         gorm:"column:email;uniqueIndex;not null"`
-	Password      string         `json:"-"             gorm:"column:password;type:text"`
-	ClienteID     *uint          `json:"cliente_id"    gorm:"column:cliente_id;index"`
-	Cliente       *Cliente       `json:"cliente"       gorm:"foreignKey:ClienteID"`
-	Rol           string         `json:"rol"           gorm:"column:rol;default:'cliente'"`
-	RoleID        *uint          `json:"role_id"       gorm:"column:role_id;index"`
-	Role          *Roles         `json:"role"          gorm:"foreignKey:RoleID"`
-	Activo         bool           `json:"activo"          gorm:"column:activo;default:true"`
-	Notas          string         `json:"notas"           gorm:"column:notas;type:text"`
-	TelegramChatID string         `json:"telegram_chat_id" gorm:"column:telegram_chat_id"`
-	Telefono       string         `json:"telefono"        gorm:"column:telefono"`
-	Pais           string         `json:"pais"            gorm:"column:pais"`
-	PortalAccesos  []PortalAcceso `json:"portal_accesos"  gorm:"foreignKey:PortalUserID"`
+	Nombre       string         `json:"nombre"        gorm:"column:nombre;not null"`
+	Email        string         `json:"email"         gorm:"column:email;uniqueIndex;not null"`
+	Password     string         `json:"-"             gorm:"column:password;type:text"`
+	ClienteID    *uint          `json:"cliente_id"    gorm:"column:cliente_id;index"`   // nil si es partner
+	Cliente      *Cliente       `json:"cliente"       gorm:"foreignKey:ClienteID"`
+	Rol          string         `json:"rol"           gorm:"column:rol;default:'cliente'"` // cliente|partner
+	Activo       bool           `json:"activo"        gorm:"column:activo;default:true"`
+	Notas        string         `json:"notas"         gorm:"column:notas;type:text"`
+	PortalAccesos []PortalAcceso `json:"portal_accesos" gorm:"foreignKey:PortalUserID"`
 }
 
 func (PortalUser) TableName() string { return "portal_users" }
@@ -48,7 +43,7 @@ func (PortalAcceso) TableName() string { return "portal_accesos" }
 func GetAllPortalUsers(limit, offset int) ([]PortalUser, int64, error) {
 	var items []PortalUser
 	var total int64
-	db := app.Http.Database.DB.Model(&PortalUser{}).Preload("Cliente").Preload("PortalAccesos.Cliente").Preload("Role")
+	db := app.Http.Database.DB.Model(&PortalUser{}).Preload("Cliente").Preload("PortalAccesos.Cliente")
 	if err := db.Count(&total).Error; err != nil {
 		return nil, 0, err
 	}
@@ -60,7 +55,7 @@ func GetAllPortalUsers(limit, offset int) ([]PortalUser, int64, error) {
 
 func GetPortalUserByID(id uint) (*PortalUser, error) {
 	var item PortalUser
-	err := app.Http.Database.DB.Preload("Cliente").Preload("PortalAccesos.Cliente").Preload("Role").First(&item, id).Error
+	err := app.Http.Database.DB.Preload("Cliente").Preload("PortalAccesos.Cliente").First(&item, id).Error
 	return &item, err
 }
 
@@ -80,12 +75,8 @@ func UpdatePortalUser(u *PortalUser) error {
 		"email":      u.Email,
 		"cliente_id": u.ClienteID,
 		"rol":        u.Rol,
-		"role_id":    u.RoleID,
-		"activo":          u.Activo,
-		"notas":           u.Notas,
-		"telegram_chat_id": u.TelegramChatID,
-		"telefono":        u.Telefono,
-		"pais":            u.Pais,
+		"activo":     u.Activo,
+		"notas":      u.Notas,
 	}).Error
 }
 
@@ -101,6 +92,41 @@ func DeletePortalUser(id uint) error {
 	return app.Http.Database.DB.Delete(&PortalUser{}, id).Error
 }
 
+// GetPortalUsersByClienteID devuelve todos los portal_users activos asociados a un cliente,
+// ya sea directamente (rol cliente) o mediante PortalAcceso (rol partner).
+func GetPortalUsersByClienteID(clienteID uint) ([]PortalUser, error) {
+	var result []PortalUser
+	db := app.Http.Database.DB
+
+	// Usuarios directos del cliente
+	var directos []PortalUser
+	if err := db.Where("cliente_id = ? AND activo = true", clienteID).Find(&directos).Error; err != nil {
+		return nil, err
+	}
+	result = append(result, directos...)
+
+	// Usuarios partner con acceso al cliente via PortalAcceso
+	var accesos []PortalAcceso
+	if err := db.Where("cliente_id = ?", clienteID).Find(&accesos).Error; err != nil {
+		return nil, err
+	}
+	seen := make(map[uint]bool)
+	for _, d := range directos {
+		seen[d.ID] = true
+	}
+	for _, a := range accesos {
+		if seen[a.PortalUserID] {
+			continue
+		}
+		var u PortalUser
+		if err := db.Where("id = ? AND activo = true", a.PortalUserID).First(&u).Error; err == nil {
+			result = append(result, u)
+			seen[u.ID] = true
+		}
+	}
+	return result, nil
+}
+
 func AddPortalAcceso(portalUserID, clienteID uint) error {
 	// Evitar duplicados
 	var existing PortalAcceso
@@ -121,11 +147,7 @@ func RemovePortalAcceso(portalUserID, clienteID uint) error {
 
 // GetClienteIDsForPortalUser devuelve todos los clienteIDs accesibles para un portal user.
 func GetClienteIDsForPortalUser(u *PortalUser) []uint {
-	isPartner := u.Rol == "partner"
-	if u.Role != nil {
-		isPartner = u.Role.EsPortalPartner
-	}
-	if isPartner {
+	if u.Rol == "partner" {
 		ids := make([]uint, 0, len(u.PortalAccesos))
 		for _, a := range u.PortalAccesos {
 			ids = append(ids, a.ClienteID)
@@ -153,12 +175,3 @@ func CheckPortalLogin(email, password string) (*PortalUser, error) {
 	}
 	return u, nil
 }
-
-// GetPortalUsersByClienteID devuelve todos los portal users activos de un cliente.
-func GetPortalUsersByClienteID(clienteID uint) ([]PortalUser, error) {
-	var users []PortalUser
-	err := app.Http.Database.DB.
-		Where("cliente_id = ? AND activo = true", clienteID).
-		Find(&users).Error
-	return users, err
-}

pkg/models/proyecto.go

@@ -124,11 +124,8 @@ func GetAllProyectos(limit, offset int, search string) ([]Proyecto, int64, error
 }
 
 func GetProyectosByClienteIDs(clienteIDs []uint) ([]Proyecto, error) {
-	if len(clienteIDs) == 0 {
-		return []Proyecto{}, nil
-	}
 	var items []Proyecto
-	err := app.Http.Database.DB.Preload("Cliente").Where("cliente_id IN ?", clienteIDs).Order("created_at DESC").Find(&items).Error
+	err := app.Http.Database.DB.Where("cliente_id IN ?", clienteIDs).Order("created_at DESC").Find(&items).Error
 	return items, err
 }
 

pkg/models/proyecto_ticket.go

@@ -9,8 +9,7 @@ import (
 
 type ProyectoTicket struct {
 	gorm.Model
-	ProyectoID   uint            `json:"proyecto_id"    gorm:"column:proyecto_id;index"`
-	Proyecto     Proyecto        `json:"proyecto"       gorm:"foreignKey:ProyectoID"`
+	ProyectoID   uint            `json:"proyecto_id"   gorm:"column:proyecto_id;index"`
 	PortalUserID uint            `json:"portal_user_id" gorm:"column:portal_user_id;index"`
 	AutorNombre  string          `json:"autor_nombre"  gorm:"column:autor_nombre"`
 	Titulo       string          `json:"titulo"        gorm:"column:titulo"`
@@ -68,13 +67,3 @@ func GetTicketsByPortalUser(portalUserID uint) ([]ProyectoTicket, error) {
 		Preload("Mensajes").Order("created_at DESC").Find(&items).Error
 	return items, err
 }
-
-func GetAllTickets(estado string) ([]ProyectoTicket, error) {
-	var items []ProyectoTicket
-	db := app.Http.Database.DB.Preload("Proyecto").Preload("Mensajes").Order("created_at DESC")
-	if estado != "" && estado != "todos" {
-		db = db.Where("estado = ?", estado)
-	}
-	err := db.Find(&items).Error
-	return items, err
-}

pkg/services/notif_dispatch.go

@@ -8,118 +8,61 @@ import (
 )
 
 // ─── DispatchTicketNuevo ──────────────────────────────────────────────────────
-// Llamar cuando el portal user crea un nuevo ticket.
-// Notifica al ADMIN según su configuración de canales.
+// Notifica al admin cuando un portal user crea un ticket.
 func DispatchTicketNuevo(ticket *models.ProyectoTicket, portalUser *models.PortalUser, proyectoNombre string) {
-	cfg := models.GetNotifConfig("ticket_nuevo", "admin")
-	if cfg == nil {
+	if ticket == nil || portalUser == nil {
 		return
 	}
-	ticketURL := "/app/tickets"
-	titulo := fmt.Sprintf("Nuevo ticket: %s", ticket.Titulo)
-	cuerpo := fmt.Sprintf("Cliente %s abrió: %s", ticket.AutorNombre, ticket.Titulo)
-
-	if cfg.CanalSistema {
-		_ = models.CreateSistemaNotif(&models.SistemaNotificacion{
-			TipoUsuario: "admin",
-			UsuarioID:   0,
-			Titulo:      titulo,
-			Cuerpo:      cuerpo,
-			Url:         ticketURL,
-			Icono:       "🎫",
-		})
-	}
-	if cfg.CanalEmail {
-		if adminEmail := getAdminEmail(); adminEmail != "" {
-			SendTicketNuevoAdmin(adminEmail, proyectoNombre, ticket.AutorNombre, ticket.Titulo, ticket.Descripcion, ticketURL)
-		}
-	}
-	if cfg.CanalTelegram {
-		msg := fmt.Sprintf("🎫 <b>Nuevo ticket</b>\nProyecto: <b>%s</b>\nCliente: %s\nTítulo: <b>%s</b>\n\n%s\n\n🔗 %s",
-			proyectoNombre, ticket.AutorNombre, ticket.Titulo, ticket.Descripcion, ticketURL)
-		sendTelegramAdmin(msg)
-	}
+	msg := fmt.Sprintf("🎫 <b>Nuevo ticket</b>\nProyecto: <b>%s</b>\nCliente: %s\nTítulo: <b>%s</b>\n\n%s",
+		proyectoNombre, ticket.AutorNombre, ticket.Titulo, ticket.Descripcion)
+	sendTelegramAdmin(msg)
 }
 
 // ─── DispatchTicketRespuestaCliente ──────────────────────────────────────────
-// Llamar cuando el portal user responde un ticket.
-// Notifica al ADMIN según su configuración de canales.
+// Notifica al admin cuando el portal user responde un ticket.
 func DispatchTicketRespuestaCliente(ticket *models.ProyectoTicket, contenido string, proyectoNombre string) {
-	cfg := models.GetNotifConfig("ticket_respuesta_cliente", "admin")
-	if cfg == nil {
+	if ticket == nil {
 		return
 	}
-	ticketURL := "/app/tickets"
-	titulo := fmt.Sprintf("Respuesta de cliente: %s", ticket.Titulo)
-	cuerpo := fmt.Sprintf("%s respondió: %s", ticket.AutorNombre, contenido)
-
-	if cfg.CanalSistema {
-		_ = models.CreateSistemaNotif(&models.SistemaNotificacion{
-			TipoUsuario: "admin",
-			UsuarioID:   0,
-			Titulo:      titulo,
-			Cuerpo:      cuerpo,
-			Url:         ticketURL,
-			Icono:       "💬",
-		})
-	}
-	if cfg.CanalEmail {
-		if adminEmail := getAdminEmail(); adminEmail != "" {
-			SendTicketRespuestaAdmin(adminEmail, proyectoNombre, ticket.AutorNombre, ticket.Titulo, contenido, ticketURL)
-		}
-	}
-	if cfg.CanalTelegram {
-		msg := fmt.Sprintf("💬 <b>Respuesta de cliente</b>\nProyecto: <b>%s</b>\nCliente: %s\n\n%s\n\n🔗 %s",
-			proyectoNombre, ticket.AutorNombre, contenido, ticketURL)
-		sendTelegramAdmin(msg)
-	}
+	msg := fmt.Sprintf("💬 <b>Respuesta de cliente</b>\nProyecto: <b>%s</b>\nCliente: %s\n\n%s",
+		proyectoNombre, ticket.AutorNombre, contenido)
+	sendTelegramAdmin(msg)
 }
 
 // ─── DispatchTicketRespuestaAdmin ─────────────────────────────────────────────
-// Llamar cuando el admin responde un ticket.
-// Notifica al PORTAL USER según su configuración de canales.
+// Notifica al portal user cuando el admin responde un ticket.
 func DispatchTicketRespuestaAdmin(ticket *models.ProyectoTicket, contenido string, proyectoNombre string) {
-	cfg := models.GetNotifConfig("ticket_respuesta_admin", "portal_user")
-	if cfg == nil {
+	if ticket == nil {
 		return
 	}
-	// Obtener portal user para saber email y telegram
 	portalUser, err := models.GetPortalUserByID(ticket.PortalUserID)
 	if err != nil || portalUser == nil {
 		log.Printf("[Notif] PortalUser %d no encontrado: %v", ticket.PortalUserID, err)
 		return
 	}
-
-	portalURL := fmt.Sprintf("/portal/proyecto/%s", ticket.Proyecto.Slug)
-	titulo := fmt.Sprintf("Respuesta en tu ticket: %s", ticket.Titulo)
-	cuerpo := fmt.Sprintf("El equipo respondió: %s", contenido)
-
-	if cfg.CanalSistema {
-		_ = models.CreateSistemaNotif(&models.SistemaNotificacion{
-			TipoUsuario: "portal_user",
-			UsuarioID:   portalUser.ID,
-			Titulo:      titulo,
-			Cuerpo:      cuerpo,
-			Url:         portalURL,
-			Icono:       "💬",
-		})
-	}
-	if cfg.CanalEmail && portalUser.Email != "" {
-		SendTicketRespuestaPortalUser(portalUser.Email, portalUser.Nombre, proyectoNombre, ticket.Titulo, contenido, portalURL)
-	}
-	if cfg.CanalTelegram && portalUser.TelegramChatID != "" {
-		ts := NewTelegramService()
-		msg := fmt.Sprintf("💬 <b>El equipo respondió tu ticket</b>\nProyecto: <b>%s</b>\nTicket: <b>%s</b>\n\n%s\n\n🔗 %s",
-			proyectoNombre, ticket.Titulo, contenido, portalURL)
-		if err := ts.SendMessageWithToken(portalUser.TelegramChatID, msg, getAdminBotToken()); err != nil {
-			log.Printf("[Notif] Error telegram portal_user %d: %v", portalUser.ID, err)
-		}
+	if portalUser.Email == "" {
+		return
 	}
+	// TODO: enviar email al portal user cuando mail_service implemente SendTicketRespuestaPortalUser
+	_ = fmt.Sprintf("/portal/dashboard")
 }
 
-// ─── helpers internos ─────────────────────────────────────────────────────────
+// ─── DispatchFacturaSubida ────────────────────────────────────────────────────
+// Notifica a los portal_users del cliente cuando se sube una factura.
+func DispatchFacturaSubida(factura *models.Factura) {
+	if factura == nil || factura.ClienteID == 0 {
+		return
+	}
+	portalUsers, err := models.GetPortalUsersByClienteID(factura.ClienteID)
+	if err != nil || len(portalUsers) == 0 {
+		return
+	}
+	log.Printf("[Notif] DispatchFacturaSubida factura=%d portal_users=%d (pendiente de implementar)",
+		factura.ID, len(portalUsers))
+}
+
+// ─── helpers ─────────────────────────────────────────────────────────────────
 
-// getAdminEmail devuelve el from_address del SMTP activo como email del admin.
 func getAdminEmail() string {
 	cfg, err := models.GetSmtpConfig()
 	if err != nil || cfg == nil {
@@ -128,7 +71,6 @@ func getAdminEmail() string {
 	return cfg.FromAddress
 }
 
-// getAdminBotToken devuelve el token del primer TelegramConfig activo.
 func getAdminBotToken() string {
 	configs, err := models.GetAllTelegramConfigs()
 	if err != nil {
@@ -142,7 +84,6 @@ func getAdminBotToken() string {
 	return ""
 }
 
-// sendTelegramAdmin envía un mensaje usando el primer TelegramConfig activo.
 func sendTelegramAdmin(mensaje string) {
 	configs, err := models.GetAllTelegramConfigs()
 	if err != nil {
@@ -159,56 +100,3 @@ func sendTelegramAdmin(mensaje string) {
 		}
 	}
 }
-// ─── DispatchFacturaSubida ────────────────────────────────────────────────────
-// Llamar cuando el admin sube el PDF de una factura.
-// Notifica a todos los portal_users activos del cliente.
-func DispatchFacturaSubida(factura *models.Factura) {
-	cfg := models.GetNotifConfig("factura_subida", "portal_user")
-	if cfg == nil {
-		return
-	}
-	if factura.ClienteID == 0 {
-		return
-	}
-	portalUsers, err := models.GetPortalUsersByClienteID(factura.ClienteID)
-	if err != nil || len(portalUsers) == 0 {
-		return
-	}
-
-	clienteNombre := ""
-	if factura.Cliente.Nombre != "" {
-		clienteNombre = factura.Cliente.Nombre
-	} else if factura.Cliente.Empresa != "" {
-		clienteNombre = factura.Cliente.Empresa
-	}
-
-	for _, u := range portalUsers {
-		u := u // capture
-		portalURL := "/portal/dashboard"
-		titulo := fmt.Sprintf("Nueva factura disponible: %s", factura.Numero)
-		cuerpo := fmt.Sprintf("Factura %s por %s %s", factura.Numero, factura.Moneda, fmt.Sprintf("%.2f", factura.Monto))
-
-		if cfg.CanalSistema {
-			_ = models.CreateSistemaNotif(&models.SistemaNotificacion{
-				TipoUsuario: "portal_user",
-				UsuarioID:   u.ID,
-				Titulo:      titulo,
-				Cuerpo:      cuerpo,
-				Url:         portalURL,
-				Icono:       "🧾",
-			})
-		}
-		if cfg.CanalEmail && u.Email != "" {
-			go SendFacturaSubidaPortalUser(u.Email, u.Nombre, clienteNombre, factura.Numero, factura.Monto, factura.Moneda, portalURL)
-		}
-		if cfg.CanalTelegram && u.TelegramChatID != "" {
-			ts := NewTelegramService()
-			msg := fmt.Sprintf("🧾 <b>Nueva factura disponible</b>\nNúmero: <b>%s</b>\nMonto: %s %.2f\n\n🔗 %s",
-				factura.Numero, factura.Moneda, factura.Monto, portalURL)
-			if err := ts.SendMessageWithToken(u.TelegramChatID, msg, getAdminBotToken()); err != nil {
-				log.Printf("[Notif] Error telegram portal_user %d: %v", u.ID, err)
-			}
-		}
-	}
-	log.Printf("[Notif] DispatchFacturaSubida factura=%d clientes notificados=%d", factura.ID, len(portalUsers))
-}