Seite 3 von 25
Re: [3.2] User reminder
Verfasst: 06.02.2020 17:33
von Dr.Death
Der
User reminder sollte auch das phpBB Datumsformat der User beachten.
Momentan gibst Du das Datum zwar dem User Format Wunsch entsprechend aus, allerdings werden die Tag und Monatsnamen nicht der gewünschten Sprache ausgegeben. ( Und die Zeitzone wird nicht berücksichtigt )
Um das entsprechend korrekt auszugeben gibt es die phpBB Funktion
user->format_date()
Code: Alles auswählen
/**
* Generate ISO 8601 date string (RFC 3339)
*/
public function format_date($time)
{
static $zone_offset;
static $offset_string;
if (empty($offset_string))
{
$zone_offset = $this->user->create_datetime()->getOffset();
$offset_string = phpbb_format_timezone_offset($zone_offset);
}
return gmdate("Y-m-d\TH:i:s", $time + $zone_offset) . $offset_string;
}
Hier ein
Vorschlag das zu ändern:
https://github.com/Mike-on-Tour/userrem ... 5d4f7acca4
Re: [3.2] User reminder
Verfasst: 07.02.2020 13:49
von Dr.Death
Mittlerweile bin ich nun fertig mit dem Code review.
Im letzten Commit von meinem PR sind nun alle Stellen bereinigt, und der EPV Check gibt auch grünes Licht.
Wenn Du magst, kann du den PR nun einfach im Gitlab prüfen und bei gefallen akzeptieren.... oder du pickst Dir nur die Kirschen heraus
https://github.com/Mike-on-Tour/userreminder/pull/1
Re: [3.2] User reminder
Verfasst: 08.02.2020 10:46
von Dr.Death
Noch eine kleiner Zusatz:
https://github.com/Mike-on-Tour/userrem ... 173c4d171a
Damit nicht "deaktivierte" Benutzer erinnert werden.... ist blöd, wenn sich ein deaktivierter Benutzer versucht anzumelden
Re: [3.2] User reminder
Verfasst: 08.02.2020 12:28
von Mike-on-Tour
Hallo Doc,
herzlichen Dank für deine Arbeit, weiß das sehr zu schätzen und lerne auch was dazu. Ich schaue mir das in den nächsten Tagen mal genauer an, dann habe ich wieder etwas mehr Zeit.
Die deaktivierten Mitglieder sind aber schon ausgeschlossen, weil bei denen 'user_inactive_reason' größer als Null ist und damit werden sie bei der Abfrage nicht betrachtet, weil hier nur Mitglieder selektiert werden, für die kein Deaktivirungs-Grund eingetragen ist.
Code: Alles auswählen
$query = 'SELECT user_id, username, user_colour, user_regdate
FROM ' . USERS_TABLE . '
WHERE user_inactive_reason = 0 /* ignore inactive users */
Mit der weiteren Zeile
Code: Alles auswählen
AND user_type <> 2 /* ignore anonymous (=== guest) and bots */
will ich tatsächlich nur verhindern, dass die Bots mit angezeigt werden.
Aber mit deinem Ansatz kann ich vielleicht beide Zeilen in einer zusammenfassen, werde mir das mal genauer ansehen, insbesondere die user_type Konstanten.
Re: [3.2] User reminder
Verfasst: 08.02.2020 15:11
von Dr.Death
Genau, die user_inactive_reason ist z.B. bei meinem deaktivierten Benutzer leer, daher selektiere ich nicht mehr danach....
Re: [3.2] User reminder
Verfasst: 08.02.2020 19:50
von Dr.Death
Damit wir zusammen aber nicht durcheinander kommen mit der "Weiter" Entwicklung, nimm den Pull Request auf Github an, oder kommentiere warum Du ihn nicht annehmen möchtest.
Solltest Du lokal weiter entwickeln und Deinen Entwicklungstand nicht weiter auf GitHub aktualisieren, werden sich unsere Entwicklungsstände weiter entfernen und später nicht mehr so einfach vereinen (mergen) können.
Solltest Du Hilfe im Umgang mit Git brauchen, stehen wir Dir gerne (auch in unserem
Discord Chat Channel) zur Verfügung.
Re: [3.2] User reminder
Verfasst: 09.02.2020 13:08
von Mike-on-Tour
Hallo Doc,
habe den Pull Request jetzt angenommen und die Dateien zusammengeführt.
Re: [3.2] User reminder
Verfasst: 09.02.2020 13:25
von Dr.Death
Habs schon gesehen, klappt ja
Vielleicht könntest Du einen "develop"-branch anlegen, um alles, was entwickelt wird, zusammenzuführen ... und wenn ein Entwicklungsschritt abgeschlossen ist, den develop branch mit dem master branch zusammenzuführen.
Re: [3.2] User reminder
Verfasst: 11.02.2020 11:28
von Dr.Death
Neuer Pull Request, hatte da einen SQL Error übersehen.
Tritt nur auf, wenn die automatische Erinnerungsfunktion aktiviert wird.
Re: [3.2] User reminder
Verfasst: 11.02.2020 16:03
von Mike-on-Tour
Dr.Death hat geschrieben: 11.02.2020 11:28
Neuer Pull Request, hatte da einen SQL Error übersehen.
Tritt nur auf, wenn die automatische Erinnerungsfunktion aktiviert wird.
Habe ich eben gemerged. Und ich glaube, mir ist es auch gelungen, eine "Develop"-Branch anzulegen.