feat(fields): #1 add sorting fields by priority field
Będę wdzięczny za zdanie. Nawiązuje to do #1 (closed)
Wprowadziłem też kilka zmian, które właściwie łamią kompatybilność PHP, ale z drugiej strony już nie wspieramy PHP 5.6, więc nie powinno być to dużym problemem (czy jednak jest?). Zresztą, zauważyłem, że w ostatnim fixie, który wrzucałem wykorzystałem ??, który jest dopiero w 7.0
Głównym powodem do podbicia PHP do wersji 7.0 na tej bibliotece była potrzeba dodania testu integracyjnego, który przy zainstalowanym PHPUnit by się nie odpalił (oczywiście, daje to też parę dodatkowych opcji, jak chociażby <=> w funkcji sortującej).
Jeśli pomysł na pole z priorytetem jest okej, ale podbicie PHP już nie, to przepiszę odpowiednio wrzucony kod.
Dodatkowo: z bootstrap.php do testów integracyjnych usunąłem zaciąganie WP i WC - o ile dokładnie przejrzałem, to ta biblioteka wykorzystuje tylko jedną funkcję z WP (sanitize_text_field).
Merge request reports
Activity
assigned to @grola, @dyszczo, @sebastian.pisula, @mateuszgbiorczyk, @potreb, and @marcinkolanko
By Bartek Jaskulski on 2021-09-02T21:24:26 (imported from GitLab)
298 298 299 299 return $this; 300 300 } 301 302 /** @return int */ 303 public function get_priority() { 304 return $this->meta['priority'] ?? 10; Chyba lepiej byłoby wartość domyślną utworzyć w konstruktorze, a tu pobierać tą, która jest zapisana w instancji klasy. Tak jak to ma miejsce w przypadku
$this->meta['class'].By Mateusz Gbiorczyk on 2021-09-03T06:35:35 (imported from GitLab)
Edited by Krzysztof Dyszczykchanged this line in version 2 of the diff
By Bartek Jaskulski on 2021-09-03T08:41:38 (imported from GitLab)
298 298 299 299 return $this; 300 300 } 301 302 /** @return int */ 303 public function get_priority() { To może, jeśli idziemy w kierunku PHP 7.0, to dorzucimy deklarację typów (
public function get_priority(): int {) zwracanych wartości?By Mateusz Gbiorczyk on 2021-09-03T06:43:16 (imported from GitLab)
Edited by Krzysztof Dyszczykchanged this line in version 2 of the diff
By Bartek Jaskulski on 2021-09-03T08:41:38 (imported from GitLab)
298 298 299 299 return $this; 300 300 } 301 302 /** @return int */ 303 public function get_priority() { 304 return $this->meta['priority'] ?? 10; 305 } 306 307 public function set_priority( int $priority ) { changed this line in version 2 of the diff
By Bartek Jaskulski on 2021-09-03T08:41:38 (imported from GitLab)
added 1 commit
- 398bb2eb - refactor: add consistency in typing and inline docs
By Bartek Jaskulski on 2021-09-03T08:41:38 (imported from GitLab)
unassigned @dyszczo
By Dyszczo on 2021-09-03T12:43:29 (imported from GitLab)