feat: add player name editing functionality in admin UI and fix site_admin access to admin dashboard
This commit is contained in:
@@ -37,7 +37,7 @@ export default async function AdminDashboard() {
|
||||
}
|
||||
|
||||
// Non-admin users should see a different dashboard
|
||||
if (user.role !== "club_admin") {
|
||||
if (user.role !== "club_admin" && user.role !== "site_admin") {
|
||||
if (user.playerId) {
|
||||
redirect("/players/" + user.playerId + "/profile")
|
||||
} else {
|
||||
|
||||
+141
-28
@@ -1,33 +1,82 @@
|
||||
import { prisma } from "@/lib/prisma"
|
||||
"use client"
|
||||
|
||||
import { useState, useEffect } from "react"
|
||||
import Navigation from "@/components/Navigation"
|
||||
import Link from "next/link"
|
||||
import { redirect } from "next/navigation"
|
||||
import { getSession } from "@/lib/auth-simple"
|
||||
import { hasRole } from "@/lib/permissions"
|
||||
|
||||
export default async function AdminPlayersPage() {
|
||||
const session = await getSession()
|
||||
|
||||
if (!session) {
|
||||
redirect("/auth/login")
|
||||
interface Player {
|
||||
id: number
|
||||
name: string
|
||||
currentElo: number
|
||||
gamesPlayed: number
|
||||
wins: number
|
||||
losses: number
|
||||
user?: {
|
||||
email: string
|
||||
} | null
|
||||
}
|
||||
|
||||
export default function AdminPlayersPage() {
|
||||
const [players, setPlayers] = useState<Player[]>([])
|
||||
const [error, setError] = useState("")
|
||||
const [editingPlayer, setEditingPlayer] = useState<Player | null>(null)
|
||||
const [newName, setNewName] = useState("")
|
||||
const [isSaving, setIsSaving] = useState(false)
|
||||
|
||||
useEffect(() => {
|
||||
fetchPlayers()
|
||||
}, [])
|
||||
|
||||
const fetchPlayers = async () => {
|
||||
try {
|
||||
const response = await fetch("/api/players")
|
||||
if (!response.ok) {
|
||||
throw new Error("Failed to fetch players")
|
||||
}
|
||||
const data = await response.json()
|
||||
setPlayers(data)
|
||||
} catch (err: unknown) {
|
||||
setError(err instanceof Error ? err.message : "Unknown error")
|
||||
}
|
||||
}
|
||||
|
||||
// Only site_admin and club_admin can manage players
|
||||
const permission = await hasRole('club_admin')
|
||||
if (!permission.allowed) {
|
||||
redirect("/auth/login")
|
||||
const handleEditClick = (player: Player) => {
|
||||
setEditingPlayer(player)
|
||||
setNewName(player.name)
|
||||
}
|
||||
|
||||
// Get all players with their user association
|
||||
const players = await prisma.player.findMany({
|
||||
include: {
|
||||
user: true,
|
||||
},
|
||||
orderBy: { name: "asc" },
|
||||
})
|
||||
const handleSaveName = async () => {
|
||||
if (!editingPlayer || !newName.trim()) return
|
||||
|
||||
// Get total count
|
||||
const playerCount = await prisma.player.count()
|
||||
setIsSaving(true)
|
||||
try {
|
||||
const response = await fetch(`/api/admin/players/${editingPlayer.id}`, {
|
||||
method: "PATCH",
|
||||
headers: {
|
||||
"Content-Type": "application/json",
|
||||
},
|
||||
body: JSON.stringify({ name: newName.trim() }),
|
||||
})
|
||||
|
||||
const data = await response.json()
|
||||
|
||||
if (data.success) {
|
||||
setPlayers(players.map(p =>
|
||||
p.id === editingPlayer.id ? { ...p, name: data.player.name } : p
|
||||
))
|
||||
setEditingPlayer(null)
|
||||
setNewName("")
|
||||
} else {
|
||||
alert(`Error: ${data.error}`)
|
||||
}
|
||||
} catch (err: unknown) {
|
||||
alert(`Error: ${err instanceof Error ? err.message : "Unknown error"}`)
|
||||
} finally {
|
||||
setIsSaving(false)
|
||||
}
|
||||
}
|
||||
|
||||
const playerCount = players.length
|
||||
|
||||
return (
|
||||
<div className="min-h-screen bg-gray-50">
|
||||
@@ -45,6 +94,12 @@ export default async function AdminPlayersPage() {
|
||||
</div>
|
||||
</div>
|
||||
|
||||
{error && (
|
||||
<div className="mb-4 p-4 bg-red-100 text-red-700 rounded">
|
||||
{error}
|
||||
</div>
|
||||
)}
|
||||
|
||||
{/* Player Table */}
|
||||
<div className="bg-white shadow rounded-lg overflow-hidden">
|
||||
<table className="min-w-full divide-y divide-gray-200">
|
||||
@@ -114,12 +169,20 @@ export default async function AdminPlayersPage() {
|
||||
)}
|
||||
</td>
|
||||
<td className="px-6 py-4 whitespace-nowrap text-sm text-gray-500">
|
||||
<Link
|
||||
href={`/players/${player.id}/profile`}
|
||||
className="text-green-600 hover:text-green-900"
|
||||
>
|
||||
View Profile
|
||||
</Link>
|
||||
<div className="flex space-x-2">
|
||||
<Link
|
||||
href={`/players/${player.id}/profile`}
|
||||
className="text-green-600 hover:text-green-900"
|
||||
>
|
||||
View
|
||||
</Link>
|
||||
<button
|
||||
onClick={() => handleEditClick(player)}
|
||||
className="text-blue-600 hover:text-blue-900"
|
||||
>
|
||||
Edit
|
||||
</button>
|
||||
</div>
|
||||
</td>
|
||||
</tr>
|
||||
))}
|
||||
@@ -128,6 +191,56 @@ export default async function AdminPlayersPage() {
|
||||
</div>
|
||||
</div>
|
||||
</main>
|
||||
|
||||
{/* Edit Player Modal */}
|
||||
{editingPlayer && (
|
||||
<div className="fixed inset-0 bg-gray-600 bg-opacity-50 overflow-y-auto h-full w-full z-50 flex items-center justify-center">
|
||||
<div className="bg-white rounded-lg shadow-xl p-6 m-4 max-w-md w-full">
|
||||
<h3 className="text-lg font-semibold text-gray-900 mb-4">
|
||||
Edit Player Name
|
||||
</h3>
|
||||
|
||||
<p className="text-sm text-gray-600 mb-4">
|
||||
Current name: {editingPlayer.name}
|
||||
</p>
|
||||
|
||||
<div className="mb-6">
|
||||
<label htmlFor="playerName" className="block text-sm font-medium text-gray-700 mb-2">
|
||||
New Name
|
||||
</label>
|
||||
<input
|
||||
type="text"
|
||||
id="playerName"
|
||||
value={newName}
|
||||
onChange={(e) => setNewName(e.target.value)}
|
||||
className="w-full px-3 py-2 border border-gray-300 rounded-md shadow-sm focus:outline-none focus:ring-green-500 focus:border-green-500"
|
||||
placeholder="Enter new player name"
|
||||
/>
|
||||
</div>
|
||||
|
||||
<div className="flex justify-end space-x-3">
|
||||
<button
|
||||
type="button"
|
||||
onClick={() => {
|
||||
setEditingPlayer(null)
|
||||
setNewName("")
|
||||
}}
|
||||
className="px-4 py-2 border border-gray-300 rounded-md shadow-sm text-sm font-medium text-gray-700 bg-white hover:bg-gray-50"
|
||||
>
|
||||
Cancel
|
||||
</button>
|
||||
<button
|
||||
type="button"
|
||||
onClick={handleSaveName}
|
||||
disabled={isSaving || !newName.trim()}
|
||||
className="px-4 py-2 border border-transparent rounded-md shadow-sm text-sm font-medium text-white bg-green-600 hover:bg-green-700 disabled:opacity-50"
|
||||
>
|
||||
{isSaving ? 'Saving...' : 'Save'}
|
||||
</button>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
)}
|
||||
</div>
|
||||
)
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user