Download Mod 5.0.16

[Christian_N]

Nee ich hab kein HTML mehr bei die Mails, lad euch doch mal ein vanilla phpBB runter schaut euch mal in der includes/usercp_register.php an wie dort EMAIL_SIG defeniert wurde nähmlich so wie ich schrieb

Code: Alles auswählen

 					$emailer->assign_vars(array(
 						'SITENAME' => $board_config['sitename'],
 						'USERNAME' => preg_replace($unhtml_specialchars_match, $unhtml_specialchars_replace, substr(str_replace("\'", "'", $username), 0, 25)),
						'EMAIL_SIG' => (!empty($board_config['board_email_sig'])) ? str_replace('<br />', "\n", "-- \n" . $board_config['board_email_sig']) : '',
  
 						'U_ACTIVATE' => $server_url . '?mode=activate&' . POST_USERS_URL . '=' . $user_id . '&act_key=' . $user_actkey)
 					);
 					$emailer->send();
 					$emailer->reset();

auszug aus der aktivierung e-mail, das str_repace() ist also vom phpBB Standard mässig schon drin, deshalb dachte ich das man es beim Download MOD auch so ggf. machen könnte.
klar kann man die <br /> Tags auch im ACP rausnehmen, dann würde im DL MOD es auch richtig anzeigen, und auch in der aktivierungs e-mail zeigt es weiterhin an wie gewollt, den da er da dann nicht zum replace hat.

Naja, oki wie gesagt war nur ein vorschlag, das es dem phpBB komform entspricht.

Hauptsache das mit dem Kategoriename ist korrigiert. :-)

[AmigaLink]

Tatsächlich. Was'n das für'n Quatsch?
Fliegt bei mir direkt mal raus!

[oxpus]

Also mal ehrlich:
Ich käme im Traum nicht auf den Gedanken, in einer Email-Signatur in einem Forum HTML zu verwenden.
Ich denke man, daß diese Ersetzungen im phpbB auch schlicht vergessen wurden herauszunehmen, da ja das phpBB pure Text-Mails versendet.
Und daran hält sich eben auch der Download MOD zu 100%.
Daher kann ich die hier aufgekommene Diskussion auch immer noch nicht verstehen. Sorry...

Und der Code fliegt bei mir jetzt auch. Was für eine Ressourcen-Verschwendung...

[AmigaLink]

Ich denke mal das phpBB-Team hatte irgendwann mal vor HTML Mails einzusetzen und deswegen diesen Code eingebaut.
Wenn man das Board mal genauer unter die Lupe nimmt. Findet man ja jede menge Code-Leichen und unsinnige Codestellen. Die schlicht und einfach vergessen wurden.
Ich habe in dieser Richtung schon so einiges Entfernt oder es aktiviert (letzteres siehe z.B. mein Useravatar in GroupCP ).

[Christian_N]

Das heißt also, das man alle *.php Dateien bearbeiten muss in denne EMAIL_SIG defeniert wurde und diese alle in

Code: Alles auswählen

'EMAIL_SIG' => $board_config['board_email_sig'],

ändern?

Hm, dann fehlt aber die 2 striche vor der Signatur, geht vllt. auch

Code: Alles auswählen

'EMAIL_SIG' => '-- \n' . $board_config['board_email_sig'],

dann, damit die 2 strichen sind und man in der Mail erkennt aha das ist eine Signatur oder ist das auch eine Ressourcen-Verschwendung dann?

Finde es nur etwas seltsam das dann auch alle MODs die eine E-Mail versendet in demfall mit Ressourcen-Verschwendung programmiert wurden u.a. auch den MOD von Oxpus "Temporary Board Users 1.0.0" den genau dort wird auch in der EMAIL_SIG diese str_replace() genutzt.

[oxpus]

In dem MOD hatte ich die Original-Funktion des Boards nur kopiert.
Ist aber auch hier kompletter Blödsinn.
Wenn Du auch dazu noch den Trennstrich weiter haben willst, dann füge ihn in der Signatur doch einfach mit ein.

Und so langsam wirds hier wirklich albern:
Die Signatur wird im Email wie im ACP eingegeben angezeigt, da wird nichts verändert. Warum dann auch noch weiter darüber diskutieren?

[Christian_N]

Oki, da steckt was dran, dann tun ich überall es in

Code: Alles auswählen

'EMAIL_SIG' => $board_config['board_email_sig'],

ändern - oki dann hat sich das ja auch erledigt

[AmigaLink]

Die Diskussion ist/war wirklich albern und noch dazu OffTopic.
Und was die Ressourcenverschwendung (in MODs) anbetrifft. Nicht jeder Autor achtet auf so etwas (insbesondere Anfänger nicht).
Meistens sind es ja nur Kleinigkeiten deren Auswirkung nicht bedacht wird. Die eMailfunktion ist da ein verdammt gutes Beispiel. Der Progger macht sich da keine Gedanken, denn wen juckt schon ein kleiner Resourcenfresser in so einer nichtigen Funktion? Was die Funktion dann aber ausbremst, wenn gerade 100 User im Board unterwegs sind oder wenn zufällig gerade 500 Mails verschickt werden müssen. Das wird schnell übersehen!

[oxpus]

Thema Massen Emails z. B.

[AmigaLink]

Nehmen wir mal den Download MOD (um wieder einen Bezug Topic zu bekommen). Oxpus.de hat derzeitig 826 Mitglieder. Die meisten davon sind hier weil sie einen oder mehrere MODs von oxpus einsetzen.
Nun gehen sie hin und aktivieren für diese MODs das eMail-Abo um über neue Versionen sofort bescheid zu wissen. Jetzt sagen wird mal das 90% der User den 'Admin User Register' Aboniert haben. Nun wird dieser aktualisiert und schwubs gehen 743 Mails raus.
743 mal wird absolut unnötig das str_replace(); ausgeführt und das wohlmöglich wenn gerade 50 User und 10 BOTs auf oxpus.de unterwegs sind.
Na Danke!

[oxpus]

Yupp und mein Email-Server quält sich dazu schon genug mit ettlichen Spam Mails rum. So an die 100+ je Tag...

[modbo]

>Anfänger
Kann mir mal jemand erläutern oder ein kleines Snippet schreiben, was da nun zu viel ist und gelöscht werden könnte?
Würde mich freuen

[oxpus]

Suche in der Datei usercp_register.php nach "EMAIL_SIG" und ersetze alle gefundenen Zeilen mit

Code: Alles auswählen

'EMAIL_SIG' => $board_config%u5b'board_email_sig'%u5d,

Auch wenn Du dieses oder ähnliche Zeilen in anderen Scripten findest, die in einem Email-Block stehen und die Board-Email-Signatur verwenden, entsprechend dieser Vorlage anpassen.

Und nun aber wieder ontopic, sonst schliesse ich hier ab!

[Patti]

So damit wir mal wieder zurück kommen zum thema

bei mir gibt es immer im ct5 immer noch diese meldung

Code: Alles auswählen

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Script-Filename: /phpBB2/downloads.php
----------------

Request-Method: GET

Possible solution:
------------------

#
#-----[ OPEN ]------------------------------------------
#
/phpBB2/downloads.php

#
#-----[ FIND ]------------------------------------------
#
include($phpbb_root_path . 'common.'.$phpEx);

#
#-----[ BEFORE, ADD ]------------------------------------------
#
define('CT_SECLEVEL', 'MEDIUM');
$ct_ignoregvar = array('');

#
#-----[ SAVE/CLOSE ALL FILES ]------------------------------------------
#
# EoM

aber bei mir steht im download.php schon so :

Code: Alles auswählen

* connect to phpBB
*/
define('IN_PHPBB', true);
$phpbb_root_path = './';
include($phpbb_root_path . 'extension.inc');
define('CT_SECLEVEL', 'LOW');
$ct_ignorepvar = array('mod_desc', 'long_desc', 'description', '');
include($phpbb_root_path . 'common.'.$phpEx);
include($phpbb_root_path . 'includes/bbcode.'.$phpEx);
include($phpbb_root_path . 'includes/functions_post.'.$phpEx);

wie man sieht habe ich extra auch schon auf LOW gestellt .
danke im vorraus für die hilfe

[oxpus]

LOW bedeutet hier nur, daß POST-Variablen komplett ignoriert werden.
GET wird aber weiterhin geprüft!
Nur: Bei was genau kommt denn dieser Alert? An welcher Stelle?
Ich hatte noch nie im Download MOD einen GET-Alert...

[Patti]

Hallo oxpus ,
das kann ich dir leider genau nicht sagen , ist nur eben zwischendurch in dne log des debug , ich versuche das gerade igrend wo zu probiren bei was das kommen soll , bisherleider ohne erfolg was die meldung ausgibt :-/
ich meine es ist vielleicht bei den kommentaren .

[oxpus]

Ich kann den Fehler immer noch nicht nachvollziehen...

[Christian_N]

Supi, habe nur bei die 2 Abonnierte DL die Mail erhalten, wie eingestellt, Update auf der neue Version ging wie immer schnell und reibungslos
Allerdings ist mir in diese E-Mail die ich bekommen habe (downloads_change_notify.tpl) aufgefallen das ein Wort fehlt:

Code: Alles auswählen

du bekommst diese Email, weil der nachfolgende Download {SITENAME} in der Kategorie "{CATEGORY}" geändert wurde:

zwischen Download und {SITENAME} mal geachtet, hört sich es nicht besser an, wenn das kleine schöne wörtchen "auf" noch dazwischen wäre also

Code: Alles auswählen

du bekommst diese Email, weil der nachfolgende Download auf {SITENAME} in der Kategorie "{CATEGORY}" geändert wurde:

Finde es jetzt nicht tragisch und hab es bei mir selbst schon angepasst so das es ggf. dann in der 5.0.18 berücksichtigt wird da es erstmal jeder Benutzer auch selbst schon vorher ändern könnte des kleine wort bevor di raus kommt irgendwann mal :-)

[Patti]

Code: Alles auswählen

Could not drop old hotlink sessions for this user

DEBUG MODE

SQL Error : 1146 Table 'usr_web75_6.phpbb_iandl_hotlink' doesn't exist

DELETE FROM phpbb_iandl_hotlink WHERE user_id = 2 

Line : 213
File : downloads.php

die meldung bekomme ich nun

wenn ich db_5_0_0_to_latest.php ausführ bekomme ich diese meldung

Code: Alles auswählen

Check Download Release 

Found: 5.0.16

Add new field to table phpbb_ianusers: user_allow_fav_download_email

+++ successfull


Add new field to table phpbb_ianusers: user_allow_fav_download_popup

+++ successfull


Insert new config value: prevent_hotlink: Yes

+++ successfull


Insert new config value: hotlink_action: Redirect to details

+++ successfull


Create new Table phpbb_iandl_hotlink

+++ successfull

Vampir Clan Foren-Übersicht 
Information 
  
Release unknown!
Installation stops here!
Please use the script "db_update.php" to install the database changes.

Unbekannte Version!
Die Installation angehalten!
Bitte verwende das Script "db_update.php", um die Änderungen in der Datenbank zu installieren 
 

edit : habe das nun mit

Code: Alles auswählen

CREATE TABLE phpbb_dl_hotlink (
       user_id MEDIUMINT(8) DEFAULT 0 NOT NULL,
       session_id VARCHAR(32) DEFAULT '' NOT NULL,
       hotlink_id VARCHAR(32) DEFAULT '' NOT NULL
);

ALTER TABLE phpbb_users ADD COLUMN user_allow_fav_download_email TINYINT(1) DEFAULT '1' NOT NULL;
ALTER TABLE phpbb_users ADD COLUMN user_allow_fav_download_popup TINYINT(1) DEFAULT '1' NOT NULL;

INSERT INTO phpbb_dl_config (config_name, config_value) VALUES ('prevent_hotlink', '1');
INSERT INTO phpbb_dl_config (config_name, config_value) VALUES ('hotlink_action', '1');

UPDATE phpbb_dl_config SET config_value = '5.0.17' WHERE config_name = 'dl_mod_version';

generator gemacht , nun klappt es .
vielleicht ein kleiner fehler in der db_update.php ? oder ich bin heute einfach nur wieder zu müde

edit 2 .

bei der templates\subSilver\dl_mod\tpl\view_dl_body.tpl
scheint sich ein fehler eingeschlichen zu haben wenn man die neue daeit nutzt ist der downlod botten weg bei den dateien.

c'ya

[oxpus]

Ich habe das Paket aktualisiert. Einfach nochmal die Dateien ersetzen.

In der db_5_0_0_to_latest.php fehlte ein break;. Ist nun auch vorhanden...

[Christian_N]

Genau das ist mir auch gerade aufgefallen der Button ist zwar da aber die Schrift "Download" fehlt in den Button oki, werds dann nochmal neu runterladen und die neue dateien hochladen - thx für die aktualisierung.

[oxpus]

BTW: Für weitere Fehler nehmt mal ein neues Topic. Der MOD ist schliesslich die 5.0.17

[AmigaLink]

Ich habe den 5.0.16 noch drin, deswegen schreibe ich nochmals hier.

Mir ist gestern aufgefallen das meine User Traffic fürs Posten bekommen, obwohl ich selbiges deaktiviert habe. Hab ich beim einbau etwas vermasselt, oder ist das ein kleiner Bug der bisher unentdeckt geblieben ist (muss auch in älteren Versionen schon drin gewesen sein)???

[oxpus]

Oha, da haben sich Klammern und die Logik komplett verabschiedet.
Ich korrigiere gleich mal den MOD.
Musst dann den Block für die functions_posts.php einfach mal ersetzen, wie ich das in der Updateanleitung beschrieben habe.
Muss dann auch die 5.0.18 sein, da der Fehler schon länger existiert...

[bce]

der fehler ist mir nicht aufgefalllen.. konnte auch nicht, da meine user traffic fürs posten bekommen