2.12 update, kritik an den update

[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

[Max]

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

[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

[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")