Vorschlag für DL-MOD 5.0.14

[Christian_N]

Hallo,
habe den DL-MOD 5.0.14 auf meine Testseite, wenn ich eine Datei hochladen will, kommt nach Absenden der Fehler

Code: Alles auswählen

Fatal error: Cannot redeclare class emailer in /www/htdocs/xxx/phpBB2/includes/emailer.php on line 28

Dieses liegt daran, weil ich u.a. dem Snippet von mir "Sende E-Mail zum Geburtstagkind" und den von Oxpus den MOD "Temporary Board Users 1.0.0" installiert habe.

Da wird in der page_header.php die emailer.php includiert schon mit include_once(); und in der dl_mod/includes/dl_upload.php wird dann mit

Code: Alles auswählen

include($phpbb_root_path . 'includes/emailer.'.$phpEx);

es nochmal gemacht, und dieses bringt dann der besagte Fehler oben, deshalb hab ich es mal in

Code: Alles auswählen

include_once($phpbb_root_path . 'includes/emailer.'.$phpEx);

geändert und der Fehler war weg.

Deshalb mein Vorschlag es in die nächste Version vllt. gleich so zu machen, da man ja nicht weiss wer noch alles diese Snippet bzw. MOD drin hat oder ggf. sogar ein anderen MOD drin hat wodurch es dann zu diese komplikation kommen könnte.

Des Weiteren möchte ich noch vorschlagen, wenn man bei der Upload-Seite bei Extern die URL ohne http://(www) eingibt das er trotzdem auf die URL nach klick des Download-Button leitet.

Also wenn man oxpus.de eingibt und jem. auf Download klickt kommt dann http://www.mydomain.de/oxpus.de was natürlich nicht gefunden werden kann dann.

Auch wenn www.oxpus.de eingegeben ist, ist desselbe, erst wenn des http:// davor ist wird auf die richtige Seite weitergeleitet.

Allerdings wird dieses dann im selben Fenster geöffnet, da fände ich deshalb auch noch den als 3. vorschlag besser, das dann es in ein neues Fenster geöffnet wird, aber nur bei Extern, bei dem Internen Download solls selbe Fenster bleiben.

Gruß Chris

[oxpus]

1. Man kann nicht alle MODs berücksichtigen, daher bleibt es bei der vorhandenen Mimik.
2. Gefährlich! Vielleicht will man nicht immer gleich einen Link haben, der funktioniert
3. Ändere ich das wieder, meckert der nächste, daß er es genau anders herum haben möchte. Denn die Ziele habe ich bereits schon mehrfach geändert (Rein in die Kartoffeln, raus aus den Kartoffeln...).

Kurzum:
Ändert es selber, dann ist jeder zufrieden.
Aber danke für die Vorschläge

[Christian_N]

1. Wenn

Code: Alles auswählen

include_once($phpbb_root_path . 'includes/emailer.'.$phpEx);

dann dürfte es ja mit andere MODs keine probleme mehr machen zumindste bei mir machts keine mehr, wenn ein andere MOD ggf. problem macht dann liegts an den MOD weil der dann ggf. mit include(); defeniert is aber nicht mehr am DL-MOD, deshalb dachte ich das es für den DL-MOD vllt. besser wäre wenn er gleich include_once(); ist.

2. Was is daran gefährlich, versteh ich nicht ganz, aba naja bleibt bei mir dann so genauso wie 3., wenn man weiss das http:// davor sein muss ist es ja auch kein größeres problem mehr

Den keine lust viel zu ändern und durch Update Datei hochladen falls sich was an der geändert hat ist alles wieder weg, nee keine lust ständig es dann neu anzupassen, nur 1. muss ich wohl oder übel dann immer anpassen bei Updates