javier
f8a1310c0f
Security fixes (27 vulnerabilities across 20 files): CRITICAL: - MediaManager: whitelist mediable types prevents RCE via class instantiation - MediaManager/OfflineSyncController: IDOR fixes, remove Auth::id()??1 fallback - ClientProjects: verify project ownership on all mutations (IDOR) - CompanyManagement: Admin role check on mount() and mutations (auth bypass) - ProjectMap: scope feature/template lookups to current project (IDOR x5) - PhaseList/TemplateManager/LayerManager: scope mutations to owned resources (IDOR) - ProjectEditTabs: Gate::authorize on mount() and updateProject() - routes/web.php: reports routes moved inside can:manage all middleware (auth bypass) MEDIUM: - layer-manager: escapeHtml() on Leaflet popup interpolations (XSS) - MediaManager: server-side MIME validation + 50MB limit - ProjectList/ProjectUsers/ProjectCompanies/PhaseProgress: auth checks added - AdminUsers/ReportsDashboard/ExportController: role/permission checks added LOW: - config/session.php: secure cookie tied to production env - OfflineSyncController: sanitize storage path (path traversal) UI integration: - project-map: Issues tab (4th) with open-count badge - project-map: project navigation bar (Dashboard/Map/Gantt/Report/Issues) - project-dashboard: action buttons for Map/Gantt/Report/Issues - project-form: validation error summary + per-field @error spans - template-manager: validation error display Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
90 lines
2.6 KiB
PHP
90 lines
2.6 KiB
PHP
<?php
|
|
|
|
namespace App\Livewire;
|
|
|
|
use Livewire\Component;
|
|
use App\Models\Project;
|
|
use App\Models\User;
|
|
use Illuminate\Support\Facades\Auth;
|
|
|
|
class ProjectUsers extends Component
|
|
{
|
|
public Project $project;
|
|
public $assignedUsers = [];
|
|
public $allUsers = [];
|
|
public $selectedUserId = '';
|
|
public $selectedRole = 'viewer';
|
|
|
|
public function mount(Project $project)
|
|
{
|
|
$user = Auth::user();
|
|
if (!$user->hasRole('Admin') && !$project->users()->where('user_id', $user->id)->exists()) {
|
|
abort(403);
|
|
}
|
|
$this->project = $project;
|
|
$this->loadUsers();
|
|
}
|
|
|
|
public function loadUsers()
|
|
{
|
|
$this->assignedUsers = $this->project->users()->withPivot('role_in_project')->get();
|
|
$assignedIds = $this->assignedUsers->pluck('id')->toArray();
|
|
$this->allUsers = User::whereNotIn('id', $assignedIds)->orderBy('name')->get();
|
|
}
|
|
|
|
public function assignUser()
|
|
{
|
|
$user = Auth::user();
|
|
if (!$user->can('assign users') && !$user->hasRole('Admin')) {
|
|
session()->flash('error', 'No tienes permisos para asignar usuarios.');
|
|
return;
|
|
}
|
|
|
|
$this->validate([
|
|
'selectedUserId' => 'required|exists:users,id',
|
|
'selectedRole' => 'required|in:supervisor,consultant,client,viewer',
|
|
]);
|
|
|
|
$this->project->users()->attach($this->selectedUserId, [
|
|
'role_in_project' => $this->selectedRole
|
|
]);
|
|
|
|
$this->reset(['selectedUserId', 'selectedRole']);
|
|
$this->loadUsers();
|
|
$this->dispatch('notify', 'Usuario asignado al proyecto.');
|
|
}
|
|
|
|
public function removeUser($userId)
|
|
{
|
|
$user = Auth::user();
|
|
if (!$user->can('assign users') && !$user->hasRole('Admin')) {
|
|
session()->flash('error', 'Sin permisos.');
|
|
return;
|
|
}
|
|
|
|
$this->project->users()->detach($userId);
|
|
$this->loadUsers();
|
|
$this->dispatch('notify', 'Usuario eliminado del proyecto.');
|
|
}
|
|
|
|
public function changeRole($userId, $role)
|
|
{
|
|
$user = Auth::user();
|
|
if (!$user->can('assign users') && !$user->hasRole('Admin')) {
|
|
session()->flash('error', 'Sin permisos.');
|
|
return;
|
|
}
|
|
if (!in_array($role, ['supervisor', 'consultant', 'client', 'viewer'])) return;
|
|
|
|
$this->project->users()->updateExistingPivot($userId, [
|
|
'role_in_project' => $role
|
|
]);
|
|
$this->loadUsers();
|
|
$this->dispatch('notify', 'Rol actualizado.');
|
|
}
|
|
|
|
public function render()
|
|
{
|
|
return view('livewire.project-users');
|
|
}
|
|
} |