Status-Abfrage "online": >= in = geändert #101
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Line 363 in c770420 Grober Fehler von mir, das sollte Alles aufwärts von 1 soll öffentlich sein, alles unterhalb nicht. Auch, um sich offen zu halten, dass es Mal einen Status 2 gibt. Analog dazu sollte auch immer das URL-Profil sein. Für mich erst nächste Woche frühestens ein Thema. |
|
OK, dann muss ein ganz anderes Problem ganz anders angegangen werden. Dann schließe ich den PR. |
christophboecker
referenced
this pull request
Sep 13, 2024
### Hinzufügen der `findByCategories`-Methode zur `Entry`-Klasse
#### Zusammenfassung:
In diesem Pull Request wird eine neue Methode `findByCategories` zur `Entry`-Klasse hinzugefügt. Diese Methode ermöglicht es, Einträge anhand einer oder mehrerer Kategorien zu finden, wobei der Status der Einträge ebenfalls berücksichtigt wird. Die Methode akzeptiert sowohl Strings als auch Arrays als Eingabeparameter für die Kategorie-IDs und ist kompatibel mit PHP 8.2.
#### Änderungen im Detail:
- **Neue Methode `findByCategories`:**
- **Parameter**:
- `string|array|null $category_ids`: Akzeptiert entweder eine kommagetrennte Liste von Kategorie-IDs als String, ein Array von Kategorie-IDs oder `null`.
- `int $status`: Definiert den Mindeststatus der Einträge, die gefiltert werden sollen.
- **Funktionalität**:
- Wenn ein String übergeben wird, wird dieser in ein Array umgewandelt.
- Die Methode verwendet `whereInList`, um die Einträge zu filtern, die den angegebenen Kategorien zugeordnet sind.
- Es wird der `status` der Einträge berücksichtigt, sodass nur Einträge zurückgegeben werden, die einen bestimmten Status erfüllen.
- **Typdeklaration**:
- Die Methode nutzt die neuen Union Types von PHP 8.2 (`string|array|null`), um klar anzugeben, welche Typen akzeptiert werden.
#### Anwendungsbeispiel:
```php
// Suche nach Einträgen, die den Kategorien 1 und 2 zugeordnet sind und den Status 1 oder höher haben
$entries = FriendsOfRedaxo\Neues\Entry::findByCategories([1, 2], 1);
// Suche mit einer kommagetrennten Liste von Kategorien als String
$entries = FriendsOfRedaxo\Neues\Entry::findByCategories('1,2', 1);
// Suche nach allen Einträgen mit einem Status von 1 oder höher, ohne Kategorienfilter
$entries = FriendsOfRedaxo\Neues\Entry::findByCategories(null, 1);
```
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
An mehreren Stellen wird wenn ich nicht gänzlich irre auf "status ist 'online'" überprüft. Allerdings nicht auf
status == 1, sondernstatus >= 1.Das ist aktuell unschädlich bei Kategorien, denn dort gibt es nur
-1(draft) und1(online). Bei den Entries ist es anders. Es gibt auch2(deleted). Eine Abfrage auf "online" würde also neben den Online-Artikeln auch die gelöschten liefern. Und Gelöschtes kann per defiition nicht online sein.neues/lib/Entry.php
Lines 499 to 505 in c770420
Der PR setzt den Vergleichsoperator auf
=statt>=bzwl öäscht ihn, da=der Default ist