(#) Sorting categories in category field by position doesn't reflect…

… changes if cache is on (Issue #25)

CHANGELOG.md

@@ -35,6 +35,7 @@ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public
 
 	(#) Unpacking of Sobi Framework in case of missing Phar support works now (Issue #80)
 	(#) Error 500 instead of 404 for undefined tasks (URL addresses) (Issue #97)
+	(#) Sorting categories in category field by position doesn't reflect changes if cache is on (Issue #25)
 
 
 ### 1.4.7.2 (8 May 2018)

Site/lib/base/cache.php

@@ -258,6 +258,8 @@ public function & cleanCategories()
 		if ( $file ) {
 			SPFs::delete( $file );
 		}
+		SPFactory::cache()
+			->addVar( null, 'categories_tree', Sobi::Section() );
 		return $this;
 	}
 

Site/lib/ctrl/adm/category.php

@@ -55,9 +55,9 @@ public function execute()
 				break;
 			case 'reorder':
 				$this->authorise( 'edit' );
+				SPFactory::cache()->cleanCategories();
 				$r = true;
 				$this->reorder();
-				SPFactory::cache()->cleanCategories();
 				break;
 			case 'up':
 			case 'down':

Site/opt/fields/category.php

@@ -435,8 +435,9 @@ private function createValues( $cats, &$result, $margin, $selector = true )
 
 	protected function loadCategories()
 	{
-		if ( ( !( $this->_cats ) || !( count( $this->_cats ) ) ) && SPLoader::path( 'etc.categories.' . Sobi::Lang( false ) . '-' . Sobi::Section(), 'front', true, 'json' ) ) {
-			$this->_cats = json_decode( SPFs::read( SPLoader::path( 'etc.categories.' . Sobi::Lang( false ) . '-' . Sobi::Section(), 'front', true, 'json' ) ), true );
+		$catFile = SPLoader::path( 'etc.categories' . Sobi::Lang( false ) . '-' . Sobi::Section(), 'front', true, 'json' );
+		if ( ( !( $this->_cats ) || !( count( $this->_cats ) ) ) && $catFile ) {
+			$this->_cats = json_decode( SPFs::read( $catFile ), true );
 		}
 		if ( !( $this->_cats ) || !( count( $this->_cats ) ) ) {
 			$this->_cats = SPFactory::cache()