*gutefrage* Komma nach einer Variabel?

[Christian_N]

Hallo,
weiss nicht ob man es so nennen kann als Titel.
Also ich habe in der Uploadseite vom Album (album_upload.php) etwas gebastelt so das nicht unter einander steht:
Berechtigt, JPG-Dateien hochzuladen:
Berechtigt, PNG-Dateien hochzuladen:
usw. und dann dahinter "Ja" bzw. "Nein" des habe ich in eine Reihe gemacht das bei mir jetzt steht
Folgende Dateiendungen sind erlaubt: JPG, PNG

Das Problem hierbei ist habe es in der lang Datei gemacht das bei JPG dann nur "JPG" steht und bei PNG hab ich ", PNG" und bei GIF hab ich ", GIF" ...

Ok eigentlich nicht schlimm da JPG eigentlich sowieso aktiviert ist, aber wenn ich "JPG" im ACP deaktivieren würden und nur PNG und GIF aktiviere würde in der Reihe stehen ", PNG, GIF" das Komma vor PNG wäre dann falsch.

Oder wenn nur GIF aktiviert ist würde ", GIF" stehen. nur wenn JPG und PNG oder GIF aktiviert ist stimmt die anzeige mit "JPG, PNG" bzw. "JPG, GIF".

So lange Rede kurzer Sinn, ich weiss das hab ich schon des öfters gesehen das man auch mit PHP irgendwie ermittel kann das ganze, nur dafür kenne ich noch nicht so den Code so das also PHP prüft ist 1 nur aktiviert? Ja = kein Komma setzen, sind 2 aktiviert? Ja = 1 Komma setzen, sind alle 3 aktiviert? Ja = 2 Kommas setzen. so das falls GIF und PNG aktiviert werden und JPG deaktiviert ist PHP es ermittelt und hinter PNG des Komma macht das er "PNG, GIF" ausgibt als wie bisher ", PNG, GIF"

Den finde es dann so einfach platzsparender...

Hoffe habe mein Problem klar genug ausgedrückt und sorry für den blöde Titel wusste einfach nicht wie es heißt und ich es nennen sollte.

Schonmal danke im voraus.

Gruß Chris (boah jetzt tut mir die finger weh von den langen Text *gg*)

[oxpus]

Warum denn in den $lang überhaupt ein Komma angeben?
Warum nicht folgendes (naja, mal ein Schnellschuss):

Code: Alles auswählen

if ($album_config['...jpg-zulassen...'])
{
	$zugelassen = ($zugelassen) ? ', '. $lang['JPG'] : $lang['JPG'];
}

if ($album_config['...png-zulassen...'])
{
	$zugelassen = ($zugelassen) ? ', '. $lang['PNG'] : $lang['PNG'];
}

if ($album_config['...gif-zulassen...'])
{
	$zugelassen = ($zugelassen) ? ', '. $lang['GIF'] : $lang['GIF'];
}

So wird nur ein Komma gesetzt, wenn vorher der Text auch schon gefüllt wurde.
$zugelassen kann man dann auch noch nachträglich mit

Code: Alles auswählen

$zugelassen = $lang['Default_zugelassen_text'] . $zugelassen;

vervollständigen, wobei

Code: Alles auswählen

$lang['Default_zugelassen_text'] = 'Folgende Dateiendungen sind erlaubt: ';

Wie auch immer das dann nochmal heisst.
Und die Bedingungen musst Du auch anpassen, da habe ich jetzt auch nur "Dummies" eingesetzt...

[Christian_N]

Alles klar, naja das war wohl eine Fehlinfo den hatte sogar wo ich eben die album_upload.php geöffnet hatte es so gehabt

Code: Alles auswählen

		'S_JPG' => ($album_config['jpg_allowed'] == 1) ? $lang['JPG'] : '',
		'S_PNG' => ($album_config['png_allowed'] == 1) ? ', ' . $lang['PNG'] : '',
		'S_GIF' => ($album_config['gif_allowed'] == 1) ? ', ' . $lang['GIF'] : '',

noch nicht mal in der $lang das Komma, aber auch das war nicht das gelbe vom EI, da auch hier wenn JPG = 0 ist dann er ", PNG" ausgegeben hatte.

Wobei das von dir mal wieder so Logik war wo man auch selbst hätte drauf kommen können.

Jedenfalls gleich verbaut und es funktioniert wie es ausschaut, habe es jetzt so gemacht, den Code von diese Beitrag oben in diesen geändert:

Code: Alles auswählen

		'S_JPG' => $jpg,
		'S_PNG' => $png,
		'S_GIF' => $gif,

und das von dir so abgeändert

Code: Alles auswählen

   	if ($album_config['jpg_allowed'])
   	{
   	   $jpg = ($png || $gif) ? ', '. $lang['JPG'] : $lang['JPG'];
	}

   	if ($album_config['png_allowed'])
   	{
   	   $png = ($jpg || $gif) ? ', '. $lang['PNG'] : $lang['PNG'];
   	}

  	if ($album_config['gif_allowed'])
  	{
  	   $gif = ($jpg || $png) ? ', '. $lang['GIF'] : $lang['GIF'];
  	}

wenn jetzt nur GIF ist steht nur "GIF" dort ohne ein Komma, wenn nur PNG steht auch nur "PNG" bei GIF und PNG stande "PNG, GIF" bei alle 3 "JPG, PNG, GIF" so wie ich dann also testen konnte funktioniert es womit es wieder das gelbe vom EI ist

Also dann danke ich dir mal für den Logiksten Code aller Zeite

Was mir allerdings aufgefallen ist, wenn man im ACP bei der Konfig alle Endung auf "Nein" stellt speichert er es und bringt noch nicht mal eine Fehlermeldung, ala das mind. eine Endung erlaubt sei müsste
.
Jetzt frag ich mich ob da ein Fehler von mir rein gekommen ist durch die ganze Änderungen am Album oder ob es ein Fehler vom Album (Smartors) selbst ist...