[3.2][3.3][RC] Userreminder

In diesem Forum können Extension-Autoren ihre Extensions vorstellen, die sich noch im Entwicklungsstatus befinden. Der Einbau in Foren im produktiven Betrieb wird nicht empfohlen.
Benutzeravatar
Dr.Death
Moderator
Moderator
Beiträge: 16103
Registriert: 23.04.2003 08:22
Wohnort: Xanten
Kontaktdaten:

Re: [3.2] User reminder

Beitrag 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

Benutzeravatar
Dr.Death
Moderator
Moderator
Beiträge: 16103
Registriert: 23.04.2003 08:22
Wohnort: Xanten
Kontaktdaten:

Re: [3.2] User reminder

Beitrag 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

Benutzeravatar
Dr.Death
Moderator
Moderator
Beiträge: 16103
Registriert: 23.04.2003 08:22
Wohnort: Xanten
Kontaktdaten:

Re: [3.2] User reminder

Beitrag 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 :-)

Mike-on-Tour
Mitglied
Beiträge: 33
Registriert: 13.01.2020 21:09

Re: [3.2] User reminder

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

Benutzeravatar
Dr.Death
Moderator
Moderator
Beiträge: 16103
Registriert: 23.04.2003 08:22
Wohnort: Xanten
Kontaktdaten:

Re: [3.2] User reminder

Beitrag von Dr.Death »

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

Benutzeravatar
Dr.Death
Moderator
Moderator
Beiträge: 16103
Registriert: 23.04.2003 08:22
Wohnort: Xanten
Kontaktdaten:

Re: [3.2] User reminder

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

Mike-on-Tour
Mitglied
Beiträge: 33
Registriert: 13.01.2020 21:09

Re: [3.2] User reminder

Beitrag von Mike-on-Tour »

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

Benutzeravatar
Dr.Death
Moderator
Moderator
Beiträge: 16103
Registriert: 23.04.2003 08:22
Wohnort: Xanten
Kontaktdaten:

Re: [3.2] User reminder

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

Benutzeravatar
Dr.Death
Moderator
Moderator
Beiträge: 16103
Registriert: 23.04.2003 08:22
Wohnort: Xanten
Kontaktdaten:

Re: [3.2] User reminder

Beitrag von Dr.Death »

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

Mike-on-Tour
Mitglied
Beiträge: 33
Registriert: 13.01.2020 21:09

Re: [3.2] User reminder

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

Antworten

Zurück zu „Extensions in Entwicklung“