Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 852
TitelFeature/ll 1192 store tags correctly
Beschreibung
Statusmerged
Created at 2023-07-17T07:03:45.138Z
Updated at 2023-07-31T06:44:14.913Z
Source branchfeature/LL-1192-store-tags-correctly
Target branchdevelop
AuthorRichard Klings
AssigneeSascha Immig

Discussion notes

Da mehrere Klassen `app()->getLocale();` verwenden: macht es nicht Sinn, das Holen der System-Language ebenfalls an einem Zentralen Ort zu machen? z.B. in einer Ffunktion in helpers.php. Z.B. so wie für den API-Key: ``` if (! function_exists('api_key')) { function api_key() { return config('api.ApiKey'); } } ``` Sascha Immig 2023-07-17T12:49:19.884Z
egal ob wir das an einen Zentralen Ort verschieben (s.o.) oder hier belassen - Die Funktion sollte `getDefaulLocale()` oder `getSystemLocale()` heißen. Sascha Immig 2023-07-17T12:50:43.260Z
resolved all discussions Sascha Immig 2023-07-17T13:12:19.114Z
Müsste nicht auch `public function testItCanBeCreated()` in `tests/Feature/Admin/TagTest.php` angepasst werden? Zeile 66/67, sowie 100/101: ``` $locale = app()->getLocale(); $this->assertDatabaseHas('tags', ["name->{$locale}" => $data['name']]); ``` Sascha Immig 2023-07-17T13:23:44.049Z
resolved all discussions Richard Klings 2023-07-18T06:38:58.827Z
added 1 commit
  • 6a657f99 - LL-1192: Die Funktion Tag::getLocale wird nun auch in dem Test und der Migration verwendet.
Compare with previous version
Richard Klings 2023-07-18T06:38:59.129Z
resolved all discussions Richard Klings 2023-07-18T09:03:53.095Z
added 1 commit
  • 393902bb - LL-1192: fixer
Compare with previous version
Richard Klings 2023-07-18T09:03:53.401Z
resolved all discussions Richard Klings 2023-07-18T10:51:32.718Z
added 1 commit
  • dc2285fe - LL-1192: Fixer
Compare with previous version
Richard Klings 2023-07-18T10:51:32.876Z
merged Sascha Immig 2023-07-31T06:44:15.006Z
mentioned in commit eaf48b75f3b52ba1a414e7eefbb48e316aea45c0 Sascha Immig 2023-07-31T06:44:15.204Z

Merge Commits

Title Id Author Created at Message
Merge branch 'develop' into feature/LL-1192-store-tags-correctly 2c88e87e22d2ceb1b850874b05a99ec40ad701ed Richard Klings 2023-07-14T11:16:05.000Z Merge branch 'develop' into feature/LL-1192-store-tags-correctly
LL-1192: fixer 393902bbcad1a71b8d3136d228eabaf7f6726ff7 Richard Klings 2023-07-18T09:03:40.000Z LL-1192: fixer
LL-1192: Tags werden nur in der Platform-Sprache bereit gestellt. Tags in… 543cdb0386930e4e11c83041f029b7f5f85c772c Richard Klings 2023-07-14T11:15:26.000Z LL-1192: Tags werden nur in der Platform-Sprache bereit gestellt. Tags in anderen Sprachen werden migriert.
LL-1192: Die Funktion Tag::getLocale wird nun auch in dem Test und der Migration verwendet. 6a657f999d9814781c0fd7fddfa3762dfa01b3c2 Richard Klings 2023-07-18T06:38:15.000Z LL-1192: Die Funktion Tag::getLocale wird nun auch in dem Test und der Migration verwendet.
LL-1192: Fixer dc2285fec5bc22b22db141de2842eadfa05629cc Richard Klings 2023-07-18T10:51:22.000Z LL-1192: Fixer