Skip to content

feat(fields): #1 add sorting fields by priority field

Krzysztof Dyszczyk requested to merge feature/add-priority-sorting into master

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