Suche

[Juliet]

Mit der Hilfe von Liu-Kang hab ich noch einen kleinen Fehler meinerseits festgestellt.

Und zwar habe ich vergessen zu erwähnen, dass in der edithome.php unter die Zeile:

$userinfo["user_pm_poptime"]= (empty($userinfo["user_pm_poptime"])) ? 0 : (int)$userinfo["user_pm_poptime"];

noch folgendes drunter muß, sonst kann der User die Option "Benachrichtigen bei neuer PN" nicht deaktivieren bzw aktivieren:

$userinfo["user_pm_mail"]= (empty($userinfo["user_pm_mail"])) ? 0 : 1;

Hoffe das wars jetzt. Bitte alle die den Code eingebaut haben nochmal prüfen ob man die Option auch aktivieren/deaktivieren kann.

[NeMeSiSX2LC]

nicht ganz eigentlich genüg nur das zweite wenn man es in der funktion save_home unterbringt.

[Juliet]

Lieber NeMeSiSX2LC,

nun erkläre nochmal bitte was Du meinst für die jenigen, die nicht zwischen den Zeilen lesen können ... wie ich zum Beispiel.

1. "Nur da zweite" -> was ist das Zweite? Die zweite Codezeile die in meinem letzten post steht? Ja, soll ja auch nur die eingefügt werden ... oder was meisnt du?

2. "in der  funktion save_home" -> ja, die Stelle die ich angegeben habe ist in der savehome

[jogi24]

hi

@ Juliet

es ist doch sehr komisch

es scheinen unterschiedliche version das VKPmX 2.1a im umlauf zu sein !!

alles deine positionen zum einfügen des codes giebt es in meiner version überhaupt nicht.

ich hab es zwar drinn, und es rennt auch, nur an anderen posionen.

also ist es fraglich, ob jeder diese code auch wirklich einbinden kann (ohne erfahrung).

[onkel]

Genau,


Ich habe es bei mir eingefügt rennen tut es nicht.

.


mfg
Onkel

[NeMeSiSX2LC]

edithome.php die funktion savehome dort mal nach dem suchen

$userinfo["noscore"] = (empty($userinfo["noscore"])) ? 0 : 1;

Darunter einfach das einfügen:

$userinfo["user_pm_mail"] = (empty($userinfo["user_pm_mail"])) ? 0 : 1;

[Juliet]

@ NeMeSiSX2LC

Und wo ist jetzt der Unterschied, ob ich es nach

$userinfo["noscore"] = (empty($userinfo["noscore"])) ? 0 : 1;

oder nach

$userinfo["user_pm_poptime"]= (empty($userinfo["user_pm_poptime"])) ? 0 : (int)$userinfo["user_pm_poptime"];

einfüge???

Beides steht in der savehome.php ... ich kann Dir irgendwie nicht folgen.
Interessant wäre hingegen das mit den unterschiedlichen edithome.php Dateien. Die Datei die mir einige User zugeschickt haben enthielten alle die user_pm_poptime Zeile ...

@ onkel

Das ist eine recht geringe Beschreibung was genau nicht funktioniert ... Kommen keine Mails an? Kann der User die Option nicht aktivieren? So weiß auch ich nicht, wo ich ansetzen soll um Dein problem zu lösen.

[Biker]

Also ich habe jetzt alles umgestrickt so wie es hier steht, aber es kommen keine Mails an!

- habe die edithome.php im YA entsprechend geändert

also hier:
   
      

Code Auswählen Erweitern

echo "<tr valign=\"top\"><td bgcolor=\"$bgcolor2\"><font class=\"content\">"._YA_PMPOPTIME1."</font></td>"
."<td bgcolor=\"$bgcolor3\" colspan=\"2\">"._YA_PMPOPTIME3." <select name=\"user_pm_poptime\">".$options."</select> <span class=\"tiny\">"._YA_PMPOPTIME2."</span></td></tr>";
}
#### hier jetzt der neue Code für die Mail-Checkbox #######
        $pn_mail = (empty($userinfo['user_pm_mail'])) ? "" : "checked";
$user_pm_mail = (empty($userinfo["user_pm_mail"]))  ? 0 : (int)$userinfo["user_pm_mail"];
echo "<tr valign=\"top\"><td bgcolor=\"$bgcolor2\"><font class=\"content\">Bei neuer PN informieren:</font></td>"
."<td bgcolor=\"$bgcolor3\" colspan=\"2\"><INPUT TYPE=\"CHECKBOX\" NAME=\"user_pm_mail\" VALUE=\"1\" ".$pn_mail."> <span class=\"tiny\">Bei einer neuen PN werden Sie vom System per Email informiert.</span></td></tr>\n";

und hier:

Code Auswählen Erweitern

function savehome($userinfo) {
global $user, $cookie, $user_prefix;

if (vkpUserIsThisTheUser($userinfo["uid"])) {
#list($userinfo["storynum"],$userinfo["commentmax"],$userinfo["uid"]) = mxForceInteger($userinfo["storynum"],$userinfo["commentmax"],$userinfo["uid"]);  // by AE
list($userinfo["storynum"],$userinfo["uid"]) = mxForceInteger($userinfo["storynum"],$userinfo["uid"]);  // by AE
$userinfo["storynum"]       = (empty($userinfo["storynum"]))   ? $GLOBALS["storyhome"]    : $userinfo["storynum"];
$userinfo["commentmax"]     = (empty($userinfo["commentmax"])) ? $GLOBALS["commentlimit"] : $userinfo["commentmax"];
$userinfo["theme"]          = (empty($userinfo["theme"]) || !@file_exists("themes/".$userinfo["theme"]."/theme.php") || !$GLOBALS["allowchangetheme"]) ? "" : $userinfo["theme"];
$userinfo["user_pm_poptime"]= (empty($userinfo["user_pm_poptime"])) ? 0 : (int)$userinfo["user_pm_poptime"];
$userinfo["user_pm_mail"]= (empty($userinfo["user_pm_mail"])) ? 0 : 1;



- habe die index.php im Modul Private_Message entsprechend geändert

Code Auswählen Erweitern

###########################
function submitmessage($arrpostdata, $uid){
include("modules/Private_Messages/inboxsize-conf.php");
global $prefix, $user_prefix, $PHP_SELF, $module_name, $refresh, $bgcolor2;
$uid = (int)$uid;
if(empty($arrpostdata["to_user"]) && empty($arrpostdata["to_all"]) || (!empty($arrpostdata["to_all"]) && !mxIsAdmin())) {
prvmsgerror(_ERRMSGNOUSER);
}
if (empty($arrpostdata["to_all"])) {
$result = sql_query("select uid from ".$user_prefix."_users where uname=\"".mxAddSlashesForSQL($arrpostdata["to_user"])."\"");
list($to_userid) = sql_fetch_row($result);
if (!$to_userid) {
prvmsgerror(_PMSUSERNOTINDB);
}
}
// Milhouse_Erweiterung
$result_anzahl = sql_query("SELECT Count(*) FROM ".$prefix."_priv_msgs WHERE to_userid=$to_userid");
$qry_sent = "SELECT Count(*) FROM ".$prefix."_priv_msgs_send WHERE from_userid=$uid";
$result_sent = sql_query($qry_sent);
$count_anzahl = sql_fetch_row($result_anzahl);
$countpmsent = sql_fetch_row($result_sent);
if ($countpmsent[0] >= $sentboxsize) {
    prvmsgerror(_SENTPMBOXVOLL);
}
if ($count_anzahl[0] >= $inboxsize) {
    prvmsgerror(_PMBOXVOLL);
}
//Ende Erweiterung
if(!($arrpostdata["subject"])) {
prvmsgerror(_ERRMSGNOSUBJECT);
}
if (strip_tags($arrpostdata["subject"]) != $arrpostdata["subject"]) {
prvmsgerror(_ERRMSGNOHTMLINSUBJECT);
}
if(!($arrpostdata["image"])) {
prvmsgerror(_ERRMSGNOICON);
}
if(!($arrpostdata["message"])) {
prvmsgerror(_ERRMSGNOMESSAGE);
}
$arrpostdata["message"] = mxAddSlashesForSQL($arrpostdata["message"]);
$arrpostdata["subject"] = mxAddSlashesForSQL(strip_tags($arrpostdata["subject"]));

$qry1 = "INSERT INTO ".$prefix."_priv_msgs (msg_image, subject, from_userid, to_userid, msg_time, msg_text) ";
$qry_msg_sent = "INSERT INTO ".$prefix."_priv_msgs_send (msg_image, subject, to_userid, from_userid, msg_time, msg_text) ";   //Gesendete PMs speichern
if (empty($arrpostdata["to_all"])) {
$cookie = mxGetUserSession();
$ignore = sql_query("select ignoreid, uname from ".$user_prefix."_users_ignorelist where uid='".$to_userid."'");
list($ignoreid, $ignorename) = sql_fetch_row($ignore);
if ($ignoreid == $uid) {

###Email bei neuer PN wenn User offline ist #####
list($touser_uname,$touser_email,$user_pm_mail) = sql_fetch_row(sql_query("SELECT uname, email, user_pm_mail FROM ".$user_prefix."_users WHERE uid='$to_userid'", $dbi), $dbi);
$message = "Halle $touser_uname,\nDu hast auf biker-unity.de eine neue Private Nachricht erhalten.
WICHTIG: Beachte, dass dies nur eine Benachrichtigung ist. Bitte antworte nicht auf diese E-Mail.
Deine Nachricht kannst Du hier abholen:\n http://www.biker-unity.de/modules.php?name=Private_Messages\n
=================================================================================
Wenn Du nicht per Email über neue PN's informiert werden willst, kannst Du die Option hier deaktivieren:\nhttp://www.biker-unity.de/modules.php?name=Your_Account&op=edithome";
$subject = "Neue private Nachricht auf biker-unity.de";
if (!mxIsUserOnline($touser_uname) && ($user_pm_mail ==1))
{
   mxMail($touser_email, "$subject", $message, $GLOBALS['adminmail'],"text","PM-Mail");
}
#######


messagesheader($uid);
echo "<table border=\"0\" cellspacing=\"1\" cellpadding=\"3\" width=\"100%\" bgcolor=\"$bgcolor1\">"
."<TR BGCOLOR=\"$bgcolor1\">"
."<TD><br><center>".$ignorename." möchste keine Nachrichten von dir!<br><a href=\"modules.php?name=$module_name\">"._PMSRETURNTOPMSG."</a></center><br /></TD>"
."</TR></table>";
}else{
$qry1 .= "VALUES ('".$arrpostdata["image"]."', '".$arrpostdata["subject"]."', ".$uid.", ".$to_userid.", now(), '".$arrpostdata["message"]."')";
  $qry_msg_sent .= "VALUES ('".$arrpostdata["image"]."', '".$arrpostdata["subject"]."', ".$to_userid.", ".$uid.", now(), '".$arrpostdata["message"]."')";
if(!$result = sql_query($qry1)) {
prvmsgerror(_ERRMSGNOSUBMIT);
}
if ($arrpostdata["notsave"]) {
        if(!$result = sql_query($qry_msg_sent)) {
prvmsgerror(_ERRMSGNOSUBMIT);
}
}
messagesheader($uid);
echo "<table border=\"0\" cellspacing=\"1\" cellpadding=\"3\" width=\"100%\" bgcolor=\"$bgcolor2\">"
."<TR BGCOLOR=\"$bgcolor2\">"
."<TD><br><center>"._PMSMSGPOSTED." ".((isset($i)) ? " (".$i." User)" : "")."<br><a href=\"modules.php?name=$module_name\">"._PMSRETURNTOPMSG."</a></center><br></TD>"
."</TR></table>";
}
}
else {
if (mxIsAdmin()) {
$i = 0;
$result = sql_query("select uid from ".$user_prefix."_users where user_stat=1 AND uid>1");
while(list($to_userid) = sql_fetch_row($result)) {
$qry2 = $qry1." VALUES ('".$arrpostdata["image"]."', '".$arrpostdata["subject"]."', ".$uid.", ".$to_userid.", now(), '".$arrpostdata["message"]."')";
$result2 = sql_query($qry2);
if ($result2) $i++;
}
}
messagesheader($uid);
echo "<table border=\"0\" cellspacing=\"1\" cellpadding=\"3\" width=\"100%\" bgcolor=\"$bgcolor2\">"
."<TR BGCOLOR=\"$bgcolor2\">"
."<TD><br><center>"._PMSMSGPOSTED." ".((isset($i)) ? " (".$i." User)" : "")."<br><a href=\"modules.php?name=$module_name\">"._PMSRETURNTOPMSG."</a></center><br></TD>"
."</TR></table>";
}

echo "<META http-equiv=\"Refresh\" content=\"".((isset($i)) ? ($refresh*4) : $refresh).";URL=modules.php?name=$module_name\">";
}

######################################################################

- habe die Zeile in die nuke_users eingefügt



- habe es mit einem ausgeloggten User getestet, bei dem ich in der DB die user_pm_mail auf "1" eingestellt habe

Alles soweit okay....nur die Mail kommt nicht!

*grübel*

Gruß
Bernd

[NeMeSiSX2LC]

Keinen sieht nur besser aufgeteilt aus wenn das eine in der edithome ist und das andere in der savehome.

Axo wozu wird das hier eigentlich gebraucht?

$user_pm_mail = (empty($userinfo["user_pm_mail"]))  ? 0 : (int)$userinfo["user_pm_mail"];

Hab nix in deinem code gefunden wo diese variable braucht.

[Juliet]

@ Biker

Du hast den Code an die falsche Stelle in der index.php eingebaut ... weiter unten in der function submitmessage gibt es nochmal ein messagesheader($uid); - und dort muß es vor!


###########################
function submitmessage($arrpostdata, $uid){
include("modules/Private_Messages/inboxsize-conf.php");
global $prefix, $user_prefix, $PHP_SELF, $module_name, $refresh, $bgcolor2;
$uid = (int)$uid;
if(empty($arrpostdata["to_user"]) && empty($arrpostdata["to_all"]) || (!empty($arrpostdata["to_all"]) && !mxIsAdmin())) {
prvmsgerror(_ERRMSGNOUSER);
}
if (empty($arrpostdata["to_all"])) {
$result = sql_query("select uid from ".$user_prefix."_users where uname=\"".mxAddSlashesForSQL($arrpostdata["to_user"])."\"");
list($to_userid) = sql_fetch_row($result);
if (!$to_userid) {
prvmsgerror(_PMSUSERNOTINDB);
}
}
// Milhouse_Erweiterung
$result_anzahl = sql_query("SELECT Count(*) FROM ".$prefix."_priv_msgs WHERE to_userid=$to_userid");
$qry_sent = "SELECT Count(*) FROM ".$prefix."_priv_msgs_send WHERE from_userid=$uid";
$result_sent = sql_query($qry_sent);
$count_anzahl = sql_fetch_row($result_anzahl);
$countpmsent = sql_fetch_row($result_sent);
if ($countpmsent[0] >= $sentboxsize) {
    prvmsgerror(_SENTPMBOXVOLL);
}
if ($count_anzahl[0] >= $inboxsize) {
    prvmsgerror(_PMBOXVOLL);
}
//Ende Erweiterung
if(!($arrpostdata["subject"])) {
prvmsgerror(_ERRMSGNOSUBJECT);
}
if (strip_tags($arrpostdata["subject"]) != $arrpostdata["subject"]) {
prvmsgerror(_ERRMSGNOHTMLINSUBJECT);
}
if(!($arrpostdata["image"])) {
prvmsgerror(_ERRMSGNOICON);
}
if(!($arrpostdata["message"])) {
prvmsgerror(_ERRMSGNOMESSAGE);
}
$arrpostdata["message"] = mxAddSlashesForSQL($arrpostdata["message"]);
$arrpostdata["subject"] = mxAddSlashesForSQL(strip_tags($arrpostdata["subject"]));

$qry1 = "INSERT INTO ".$prefix."_priv_msgs (msg_image, subject, from_userid, to_userid, msg_time, msg_text) ";
$qry_msg_sent = "INSERT INTO ".$prefix."_priv_msgs_send (msg_image, subject, to_userid, from_userid, msg_time, msg_text) ";   //Gesendete PMs speichern
if (empty($arrpostdata["to_all"])) {
$cookie = mxGetUserSession();
$ignore = sql_query("select ignoreid, uname from ".$user_prefix."_users_ignorelist where uid='".$to_userid."'");
list($ignoreid, $ignorename) = sql_fetch_row($ignore);
if ($ignoreid == $uid) {
messagesheader($uid);
echo "<table border=\"0\" cellspacing=\"1\" cellpadding=\"3\" width=\"100%\" bgcolor=\"$bgcolor1\">"
."<TR BGCOLOR=\"$bgcolor1\">"
."<TD><br><center>".$ignorename." möchste keine Nachrichten von dir!<br><a href=\"modules.php?name=$module_name\">"._PMSRETURNTOPMSG."</a></center><br /></TD>"
."</TR></table>";
}else{
$qry1 .= "VALUES ('".$arrpostdata["image"]."', '".$arrpostdata["subject"]."', ".$uid.", ".$to_userid.", now(), '".$arrpostdata["message"]."')";
  $qry_msg_sent .= "VALUES ('".$arrpostdata["image"]."', '".$arrpostdata["subject"]."', ".$to_userid.", ".$uid.", now(), '".$arrpostdata["message"]."')";
if(!$result = sql_query($qry1)) {
prvmsgerror(_ERRMSGNOSUBMIT);
}
if ($arrpostdata["notsave"]) {
        if(!$result = sql_query($qry_msg_sent)) {
prvmsgerror(_ERRMSGNOSUBMIT);
}
}

###Email bei neuer PN wenn User offline ist #####
list($touser_uname,$touser_email,$user_pm_mail) = sql_fetch_row(sql_query("SELECT uname, email, user_pm_mail FROM ".$user_prefix."_users WHERE uid='$to_userid'", $dbi), $dbi);
$message = "Halle $touser_uname,\nDu hast auf biker-unity.de eine neue Private Nachricht erhalten.
WICHTIG: Beachte, dass dies nur eine Benachrichtigung ist. Bitte antworte nicht auf diese E-Mail.
Deine Nachricht kannst Du hier abholen:\n http://www.biker-unity.de/modules.php?name=Private_Messages\n
=================================================================================
Wenn Du nicht per Email über neue PN's informiert werden willst, kannst Du die Option hier deaktivieren:\nhttp://www.biker-unity.de/modules.php?name=Your_Account&op=edithome";
$subject = "Neue private Nachricht auf biker-unity.de";
if (!mxIsUserOnline($touser_uname) && ($user_pm_mail ==1))
{
   mxMail($touser_email, "$subject", $message, $GLOBALS['adminmail'],"text","PM-Mail");
}
#######

messagesheader($uid);
echo "<table border=\"0\" cellspacing=\"1\" cellpadding=\"3\" width=\"100%\" bgcolor=\"$bgcolor2\">"
."<TR BGCOLOR=\"$bgcolor2\">"
."<TD><br><center>"._PMSMSGPOSTED." ".((isset($i)) ? " (".$i." User)" : "")."<br><a href=\"modules.php?name=$module_name\">"._PMSRETURNTOPMSG."</a></center><br></TD>"
."</TR></table>";
}
}
else {
if (mxIsAdmin()) {
$i = 0;
$result = sql_query("select uid from ".$user_prefix."_users where user_stat=1 AND uid>1");
while(list($to_userid) = sql_fetch_row($result)) {
$qry2 = $qry1." VALUES ('".$arrpostdata["image"]."', '".$arrpostdata["subject"]."', ".$uid.", ".$to_userid.", now(), '".$arrpostdata["message"]."')";
$result2 = sql_query($qry2);
if ($result2) $i++;
}
}
messagesheader($uid);
echo "<table border=\"0\" cellspacing=\"1\" cellpadding=\"3\" width=\"100%\" bgcolor=\"$bgcolor2\">"
."<TR BGCOLOR=\"$bgcolor2\">"
."<TD><br><center>"._PMSMSGPOSTED." ".((isset($i)) ? " (".$i." User)" : "")."<br><a href=\"modules.php?name=$module_name\">"._PMSRETURNTOPMSG."</a></center><br></TD>"
."</TR></table>";
}
echo "<META http-equiv=\"Refresh\" content=\"".((isset($i)) ? ($refresh*4) : $refresh).";URL=modules.php?name=$module_name\">";
}
######################################################################

Wenn Du es dort einfügst und natürlich die Option unter "Deine Einstellungen" aktivierst müßte es gehen.

@ NeMeSiSX2LC

Hast Recht, die eine Zeile ist ein Überbleibsel aus dem alten Codegewusel und findet ja später in der function savehome Platz. Kann man also streichen.

[NeMeSiSX2LC]

Puh dacht schon ich hab was übersehen. Hab mir hier ein Wolf gesucht:-)

[Biker]

@Juliet

Bist ein Schatz :-*...und hattest natürlich Recht!   
Vielen Dank...jetzt funzt es! 

Lieben Gruß
Bernd

[Musicman75]

Vielen Dank Juliet, funktioniert alles einwandfrei - und jetzt das berühm-brüchtigte "ABER"

wie kann man nen Zeilenumbruch in die Mail einfügen, die die User dann bekommen?

Und dann fällt mir gerade noch ne Funktion ein - kann man das irgendwie machen, das der User nur bei der ersten PM ne mail bekommt und dann erst wieder eine, wenn er in seinem Account war? Wenn jetzt nen Uder viele PMs bekommt, wird der ja mit mails bombadiert.

Gruß
Steff

[Juliet]

@ Biker: freut mich 

@ Steff
Umbrüche machst Du mit \n

Für Deinen Funktionswunsch müßte man dann aber abspeichern, dass bereits eine Mailbenachrichtigung gesendet wurde - besucht der User seinen Account muß der Wert wieder auf 0 gesetzt werden etc ... Grundsätzlich gilt: geht nicht - gibt's nicht, aber ich halte diese Funktion für nicht sehr sinnvoll. Ein User der sowieso viele PNs bekommt wird wohl auch die Seite regelmäßig besuchen und brauch daher die PN-Benachrichtung eigentlich gar nicht. Außerdem soll ihn die Mail ja gerade wieder auf die Seite "locken" ...

[Musicman75]

Das mit dem \n als Zeilenumbruch habe ich auch schon probiert, bringt aber nichts.

Bei mir im Outlook werden die Mails als fortlaufender Text angezeigt ohne jeden Zeilenumbruch.

[Biker]

Das dürfte dann wohl eher ein Problem von den Einstellungen im Outlook sein!

Bei mir kommen die Mails nämlich per Outlook im html-Format MIT Zeilenumbruch an!

Empfängst Du die Nachricht als html oder "nur Text"
....wenn letzeres zutrifft, sieh mal nach, ob Du bei Dir im Outlook

unter Optionen->Einstellungen->E-Mail-Optionen

bei Nachrichtenbehandlung das Häkchen bei "Zusätzliche Zeilenumbrüche in nur Text-Nachrichten entfernen" stehen hast?

Wenn ja, mach es mal raus, und versuch es nochmal.....

[Musicman75]

Habe ich schon gemacht, werden aber immer noch ohne Zeilenumbrüche angezeigt.

[Juliet]

Hi Steff,

nimm aus der Zeile
mxMail($touser_email, "$subject", $message, $GLOBALS['adminmail'],"text","PM-Mail");
einfach mal das "text" raus und guck wie Dein Outlook das dann darstellt

[jubilee]

Hallo !

nimm aus der Zeile
mxMail($touser_email, "$subject", $message, $GLOBALS['adminmail'],"text","PM-Mail");

Das ist nutzlos. Wenn dieses Feld nicht belegt ist, wird automatisch als text-Mail versendet.
Wenn text oder nix angegeben ist, wird automatisch eine Text-EMail erstellt.
Alternative ist nur, hier anstatt text      html anzugeben.

MfG
jubilee

[Biker]

Versteh ich nicht...bei mir steht dort auch "text" drin, aber die Mail kommt im html-Format an...seltsam!?