[3.2][3.3][CDB] Userreminder

[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

[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

[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

[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.

[Dr.Death]

Genau, die user_inactive_reason ist z.B. bei meinem deaktivierten Benutzer leer, daher selektiere ich nicht mehr danach....

[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.

[Mike-on-Tour]

Hallo Doc,
habe den Pull Request jetzt angenommen und die Dateien zusammengeführt.

[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.

[Dr.Death]

Neuer Pull Request, hatte da einen SQL Error übersehen.
Tritt nur auf, wenn die automatische Erinnerungsfunktion aktiviert wird.

[Mike-on-Tour]

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.