[php] html showen alleen voor de HoofdAdmin

[Zakkenwasser]

Ik heb 4 gebruikers die kunnen inloggen op mijn website in aanmaak.

Om functie's met betrekking van gebruikers in eigen handen te houden, wil ik alleen voor mijn inlognaam een link tevoorschijn laten komen.

Ik haal daarbij de gebruikers_id / gebruikers_naam uit de sessie.

Wat ik zelf al heb geprobeerd:

[code=php]
$_SESSION['username'] = $admin;
if ($admin == 'jey');
{
echo "<p><a href=\"new_user.php\">Nieuwe gebruiker</a></p>";
echo "<br />";
echo "<hr />";
}
[/code=php]

waardoor de link nog steeds zichtbaar is voor iedereen!

Dit heb ik ook geprobeerd :

[code=php]
if ($_SESSION['user_id'] = 1 {
echo "<p><a href=\"new_user.php\">Nieuwe gebruiker</a></p>";
echo "<br />";
echo "<hr />";
}
[/code=php]

Wat ook het zelfde resultaat geeft.

[Trinsec]

Eh, is het niet andersom?
Je deed: $_SESSION['username'] = $admin;, maar is het niet eerder $admin = $_SESSION['username']; ?

Ik heb het idee dat je PHP nog niet echt goed beheerst.
if ( $_SESSION['user_id'] = 1 { ... zou liever moeten zijn: if ( $_SESSION['user_id'] == 1 ) { ... Anders ga je $_SESSION['user_id'] altijd 1 maken op die manier.

[vdwijngaert]

PHP:

$admin = $_SESSION['username'];
if ($admin == 'jey')
{
echo "<p><a href=\"new_user.php\">Nieuwe gebruiker</a></p>";
echo "<br />";
echo "<hr />";
}

Dit al geprobeerd?

[Stoffel]

Daarbij is het natuurlijk niet heel erg veilig om een link gewoon te verbergen, ik zou in die new_user.php ook maar even checken of de huidige gebruiker de goede rechten heeft.

[Zakkenwasser]

Trinsec schreef op woensdag 31 december 2008 @ 15:33:
Eh, is het niet andersom?
Je deed: $_SESSION['username'] = $admin;, maar is het niet eerder $admin = $_SESSION['username']; ?

Ik heb het idee dat je PHP nog niet echt goed beheerst.
if ( $_SESSION['user_id'] = 1 { ... zou liever moeten zijn: if ( $_SESSION['user_id'] == 1 ) { ... Anders ga je $_SESSION['user_id'] altijd 1 maken op die manier.

Die had ik dus idd ook geprobeerd

want
== gelijk aan
en
= wordt

[code=php]
<?php
if ($_SESSION['user_id'] == 1)
{
echo "<p><a href=\"new_user.php\">Nieuwe gebruiker</a></p>";
echo "<br />";
echo "<hr />";
}
?>
[/code=php]

o wacht, ik was nog ingelogd, het werkt nu gewoon, bedankt voor de moeite

[Icekiller2k6]

PHP:

<?php
if ($_SESSION['user_id'] = 1 {
echo "<p><a href=\"new_user.php\">Nieuwe gebruiker</a></p>";
echo "<br />";
echo "<hr />";
}
?>

is een grapje zeker?
het moet:

PHP:

<?php
if ($_SESSION['user_id'] == 1 {
echo "<p><a href=\"new_user.php\">Nieuwe gebruiker</a></p>";
echo "<br />";
echo "<hr />";
}
?>

zie je het verschil? dubbele =
ander stel je gewoon de session user id gelijk aan 1...

moet leren f5 te duwen

[ Voor 8% gewijzigd door Icekiller2k6 op 31-12-2008 15:39 ]

[Cartman!]

Bedenk alleen wel dat het niet echt uitbreidbaar is zo. Als er nog iemand die link mag zien kun je elke controle handigmatig gaan aanpassen.

[Teeno]

Ik heb geen verstand van PHP verder hoor, maar neem aan dat je die users in een database hebt staan. Waarom voeg je aan je usertabel geen isAdmin boolean veld toe? Dan kan je het per user toch beheren en hoef je bij uitbreiding niet in de code te gaan rommelen.

[Verwijderd]

Icekiller2k6 schreef op woensdag 31 december 2008 @ 15:38:

PHP:

<?php
if ($_SESSION['user_id'] = 1 {
echo "<p><a href=\"new_user.php\">Nieuwe gebruiker</a></p>";
echo "<br />";
echo "<hr />";
}
?>

is een grapje zeker?
het moet:

PHP:

<?php
if ($_SESSION['user_id'] == 1 {
echo "<p><a href=\"new_user.php\">Nieuwe gebruiker</a></p>";
echo "<br />";
echo "<hr />";
}
?>

zie je het verschil? dubbele =
ander stel je gewoon de session user id gelijk aan 1...

moet leren f5 te duwen

Is een grapje zeker?
het moet:

PHP:

<?php
if ($_SESSION['user_id'] == 1)
{
    echo "<p><a href=\"new_user.php\">Nieuwe gebruiker</a></p>";
    echo "<br />";
    echo "<hr />";
}
?>

zie je het verschil? een ) op het einde
ander krijg je gewoon een parse error...

[Zakkenwasser]

Teeno schreef op donderdag 01 januari 2009 @ 19:03:
Ik heb geen verstand van PHP verder hoor, maar neem aan dat je die users in een database hebt staan. Waarom voeg je aan je usertabel geen isAdmin boolean veld toe? Dan kan je het per user toch beheren en hoef je bij uitbreiding niet in de code te gaan rommelen.

True, maar ik ben nog niet eens klaar met CMS v1 klaar te maken.

Verder verbeteringen als ik tegen herhalingen aanloop pas ik dan wel aan.
En natuurlijk ook handig voor het leer process en het verder dromen van php structuur.

[ Voor 3% gewijzigd door Zakkenwasser op 01-01-2009 20:35 ]

[MueR]

Maar als je het op deze manier doet weet je al dat het niet toereikend gaat zijn. Waarom ga je dan koppig verder?

[Patriot]

MueR schreef op donderdag 01 januari 2009 @ 20:47:
Maar als je het op deze manier doet weet je al dat het niet toereikend gaat zijn. Waarom ga je dan koppig verder?

Als het voor hem persoonlijk is, waarom moet iedere ervaren GoT'er dan per sé hem ervan overtuigen dat dit verkeerd is?

Het is verre van de beste (of überhaupt een goede) aanpak, maar naast leren is simpelweg aankloten ook nog wel eens handig.

offtopic:
Is overigens geen opmerking aan het adres van jouw persoonlijk hoor MueR. Het valt me gewoon in het algemeen op dat men iedereen wil overtuigen. Ik zag het jou nu doen, dus greep mijn kans om er wat van te zeggen.

[PainkillA]

ga gewoon een database gebruiken met bj elke user een varabelle met het toegangsniveau. Vervolgens in elke pagina checks bouwen met je gewenste toegang niveau.

[RobIII]

PainkillA schreef op donderdag 01 januari 2009 @ 21:38:
ga gewoon een database gebruiken met bj elke user een varabelle met het toegangsniveau. Vervolgens in elke pagina checks bouwen met je gewenste toegang niveau.

Kijk dan liever naar een uitbreidbaarder en flexibeler systeem als RBAC. Met een paar tabelletjes en een paar simpele queries zet je een compleet flexibel rechtensysteem op en daar heb ik in zowat elk project profijt van.