[BroatcasT]

Ähhh.... ja...
war gerade ein wenig am stöbern gewesen ... und bin auf mysteriöse Dinge gestoßen.

Was hier wohl im den Köpfen der Programmierer passierte?

File: class_plugins.php Line: 99

PHP-Code:

$oldreturnargs = $returnargs; // why is this line of code here? 


Nach kurzer Recherche kann ich auch definitiv behaupten, es hat keinerlei Verwendung oder Sinn.



Edit: Also ich für meinen Teil hab Betreffende Zeile entfernt

[Zwoetzen]

Solche Dinge sind meist Altlasten, die Update für Update mitgeschleift werden, da der frühere Zusammenhang mitlerweile nicht mehr bekannt ist. Anstatt die Zeile nun einfach zu löschen, und dabei das Risiko einzugehen, dass sie doch für irgendwas wichtig war, lässt man sie dann wohl doch lieber drin ^^

Der beste Weg wäre wohl, dies direkt im BugTracker des MyBB-Team zu melden, sodass sie die Zeile vollständig mit dem nächsten Update entfernen können (Ansonsten wirst du die Zeile mit dem nächsten Update wieder drin haben ^^)

[StefanT]

Es ist wohl fast unmöglich herauszufinden, wer das so gemacht hat. Es ist sicher nicht die einzigste Kuriosität...

[frostschutz]

Theoretisch kannst ja im SVN nachschauen, wenn die History so weit zurückreicht.

Andererseits ist SVN so gähnend langsam daß man es dann doch lieber wieder läßt.

Leider ist die Beurteilung bei MyBB auch recht schwer, was solche Variablen angeht. An der Stelle jetzt vielleicht nicht, aber MyBB hat ja Variablen fast ausschließlich im globalen Scope. D.h. selbst wenn MyBB selbst da irgendwas nicht verwendet, kann es durchaus ein Template oder Plugin geben, in dem es dann eben doch gebraucht wird.

Es ist zum Glück ja auch recht harmlos - gibt schlimmere Stellen in MyBB als diese.

[NetHunter]

..............
Es ist zum Glück ja auch recht harmlos - gibt schlimmere Stellen in MyBB als diese.

Psssst.....

[Zwoetzen]

Leider ist die Beurteilung bei MyBB auch recht schwer, was solche Variablen angeht. An der Stelle jetzt vielleicht nicht, aber MyBB hat ja Variablen fast ausschließlich im globalen Scope. D.h. selbst wenn MyBB selbst da irgendwas nicht verwendet, kann es durchaus ein Template oder Plugin geben, in dem es dann eben doch gebraucht wird.

Ich habe mir die Stelle jetzt mal angeschaut. Die betreffende Funktion run_hooks() verwendet keine globalen Variablen, dh. spätestens am Ende der Funktion wird der Wert von $oldreturnargs ungültig. $oldreturnargs wird auch an keiner anderen Stelle innerhalb der ~30 Zeilen langen Funktion verwendet.

Innerhalb der Funktion werden die Plugin-Hooks ausgeführt, allerdings dürften die Plugin-Funktionen $oldreturnargs nicht sehen können, da es ja neue FUnktionen sind und der Scope somit neu aufgebaut wird.
Templates werden innerhalb der Funktion keine aufgerufen, somit kann auch dort $oldreturnargs nicht verwendet werden.

Also für mich sieht die Zeile ebenfalls überflüssig aus und dürfte ohne Nebenwirkungen entfernbar sein ^^

[Michael]

Ticket dazu: http://dev.mybb.com/issues/1383

[BroatcasT]

Hab mal herum gebuddelt ... das könnt ihr besser hier als in der SVN nachsehen... (auch wenn Radmine auch nur ne SVN ist wenn man's im groben betrachtet) .

Also bemerkt wurde es von "Ryan Gordon" und wurde in der File Revision "2664" als Kommentar hinzugefügt. Das ganze bereits in der Version 1.4.0.
Jetzt kommt's aber noch härter ^^.
Mitgeschliffen wird diese Tote Variable Seit der Version 1.0 ^^, an die Pre und Alpha Releases komm ich hier gerade nicht ran (doofe Arbeit) aber die habe ich zu Hause sicherlich noch irgendwo auf meinem Rechner liegen dann könnte man auch mal nachvollziehen woher dieser Spuk denn jetzt kommt.....

Long long way.....

...Es ist zum Glück ja auch recht harmlos - gibt schlimmere Stellen in MyBB als diese. ...

So etwas darfst du doch nicht laut sagen xD.

...Leider ist die Beurteilung bei MyBB auch recht schwer, was solche Variablen angeht. ...

Falsch guckst du ->hier<-
Edit: (schau dir hier mal dieses kleine aber unscheinbare Search oben rechts an)
Thx @ micha für diesen geilen Service

...Es ist wohl fast unmöglich herauszufinden, wer das so gemacht hat...

Falsch


Last but not least
@ mischaaa
thx fürs Ticket

[frostschutz]

Falsch guckst du ->hier<-

[ ] Du hast verstanden, was ich gemeint habe...

[StefanT]

Falsch

Nicht unbedingt, je nachdem, wann das eingefügt wurde. Das SVN reicht ja nicht ewig zurück, davor wird es dann ganz schwer. Und du hast ja gesehen, dass es wirklich lange im Code ist.