Seite 1 von 1

2.12 update, kritik an den update

Verfasst: 26.02.2005 03:45
von SevenIsMy
wieso wurden in 2.11 und 2.12 neue feathers eingebaut? visual confirm in 2.11 und in 2.12 update reminder, ich verstehe das es ja spass macht neue feathers einzubauen und das sie nicht ganz unsinnvoll sind aber dann bitte in einem mod und nicht in einen bugfix release!
und wie wers wenn die bugfixes etwas länger getestet werden, in 2.12 musste ich schon an zwei sachen patchen die schon in 2.11 gepatch wurden
außerdem was ist das bitte:

Code: Alles auswählen

#
#-----[ FIND ]---------------------------------------------
# Line 462
			$avatar_mode = ( !empty($user_avatar_name) ) ? 'local' : 'remote';

#
#-----[ REPLACE WITH ]---------------------------------------------
# 
			$avatar_mode = (empty($user_avatar_name)) ? 'remote' : 'local';
sowas kann man sich echt sparen

gut ich habe nicht für das phpbb bezahlt, aber ich verlange ja keine aufwand sondern nur etwas verstädnis für admins die so wenig wie möglich zeit verbringen wollen mit patchen

ps. ich weiss das es hier nicht die richtigen erreicht, ich will mir nur etwas luft machen

Re: 2.12 update, kritik an den update

Verfasst: 26.02.2005 07:41
von Max
SevenIsMy hat geschrieben: außerdem was ist das bitte:

Code: Alles auswählen

#
#-----[ FIND ]---------------------------------------------
# Line 462
			$avatar_mode = ( !empty($user_avatar_name) ) ? 'local' : 'remote';

#
#-----[ REPLACE WITH ]---------------------------------------------
# 
			$avatar_mode = (empty($user_avatar_name)) ? 'remote' : 'local';
sowas kann man sich echt sparen
Könnte wohl hiermit >Klick!< zu tun haben.

Ansonsten ist dies nun wirklich kein aufregend kompliziertes Update gewesen.


Gruß Max

Verfasst: 26.02.2005 08:37
von Garnele
Hi
außerdem was ist das bitte:

Code: Alles auswählen

# 
#-----[ FIND ]--------------------------------------------- 
# Line 462 
         $avatar_mode = ( !empty($user_avatar_name) ) ? 'local' : 'remote'; 

# 
#-----[ REPLACE WITH ]--------------------------------------------- 
# 
         $avatar_mode = (empty($user_avatar_name)) ? 'remote' : 'local'; 
sowas kann man sich echt sparen
Soweit ich weiss ganz und gar nicht. Das ! bedeutet in PHP nicht, oder ungleich (Bsp. != = ungleich). We du siehst erhält die Funkton also eine ganz Auswertung. Dass es mit der Sicherheitslücke zutun hat kann ich mir auch vorstellen.

LG
Garnele

Verfasst: 26.02.2005 12:23
von eiskuh
Was das ist? Na nen Code-Rework, um die Leserlichkeit zu erhöhen. Das ist eine unnötige Negierung und der neue Ausdruck bewirkt das gleiche bei leichterer Verständlichkeit. ("nimm a wenn leer, sonst b" ist klarer als "nimm b wenn nicht leer, sonst a")