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

src/Field/BasicField.php

@@ -3,6 +3,7 @@
 namespace WPDesk\Forms\Field;
 
 use WPDesk\Forms\Field;
+use WPDesk\Forms\Form\FormWithFields;
 use WPDesk\Forms\Sanitizer\NoSanitize;
 use WPDesk\Forms\Serializer;
 use WPDesk\Forms\Serializer\NoSerialize;
@@ -25,6 +26,7 @@ abstract class BasicField implements Field {
 
 	public function __construct() {
 		$this->meta['class'] = [];
+		$this->meta['priority'] = 10;
 	}
 
 	public function get_label() {
@@ -298,4 +300,19 @@ abstract class BasicField implements Field {
 
 		return $this;
 	}
+
+	public function get_priority(): int {
+		return $this->meta['priority'];
+	}
+
+	/**
+	 * Fields are sorted by lowest priority value first, when getting FormWithFields
+	 *
+	 * @see FormWithFields::get_fields()
+	 */
+	public function set_priority( int $priority ): self {
+		$this->meta['priority'] = $priority;
+
+		return $this;
+	}
 }

[Krzysztof Dyszczyk]

Może domyślna wartość do stałej w klasie?

By Mateusz Gbiorczyk on 2021-09-03T12:41:33 (imported from GitLab)

[Krzysztof Dyszczyk]

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)

[Krzysztof Dyszczyk]

Stanąłem w dziwnym rozkroku między 5.6 a 7.0. Dodałem typ :)

By Bartek Jaskulski on 2021-09-03T08:43:05 (imported from GitLab)

[Krzysztof Dyszczyk]

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)

[Krzysztof Dyszczyk]

Racja, to lepsze, niż sprawdzanie czy pole jest nullem

By Bartek Jaskulski on 2021-09-03T08:42:13 (imported from GitLab)

[Krzysztof Dyszczyk]

Może by tu dorzucić komentarz, że priorytety są obsługiwane rosnąco? Tak dla pewności, bo priorytet może wskazywać na to, że im wyższa wartość tym wyższe miejsce w kolejności.

By Mateusz Gbiorczyk on 2021-09-03T12:43:52 (imported from GitLab)

[Krzysztof Dyszczyk]

Dorzuciłem odpowiedni komentarz :)

By Bartek Jaskulski on 2021-09-03T08:43:22 (imported from GitLab)