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)