WIP allow to place custom-fields in tabs: fix automatic extension and use name not label

name for filename type cf is a relative path, optionally ending in a / to create a directory with all files the user uploads preserving their name

api/js/etemplate/et2_extension_customfields.ts

@@ -880,12 +880,6 @@ export class et2_customfields_list extends et2_valueWidget implements et2_IDetac
 		attrs.type = 'vfs-upload';
 		delete(attrs.label);
 
-		// fix vfs-upload id="$app:$id:$rest" e.g. "infolog:$cont[info_id]:test.pdf"
-		const vfs_app_id_regexp = /^#([a-z_]+):([@$][^:]+):(.*)$/;
-		if (attrs.id.match(vfs_app_id_regexp))
-		{
-			attrs.id = <string>this.getArrayMgr('content').expandName(attrs.id);
-		}
 		// allow to set/pass further et2_file attributes to the vfs-upload
 		if (field.values && typeof field.values === 'object')
 		{

api/src/Etemplate/Widget/Customfields.php

@@ -343,7 +343,7 @@ class Customfields extends Transformer
 			case 'vfs-upload':
 				$widget->attrs['path'] = $field['app'] . ':' .
 					self::expand_name('$cont['.Api\Link::get_registry($field['app'],'view_id').']',0,0,0,0,self::$request->content).
-					':'.$field['label'];
+					':'.(preg_match('/^[a-z_]+:[^:]+:(.+)$/', $field['name'], $matches) ? $matches[1] : $field['name']);
 				break;
 
 			case 'link-to':

api/src/Etemplate/Widget/Vfs.php

@@ -39,7 +39,7 @@ class Vfs extends File
 	{
 		if ($this->type === 'vfs-upload' || !empty($this->attrs['type']) && $this->attrs['type'] === 'vfs-upload')
 		{
-			$form_name = self::form_name($cname, $this->id, $expand ? $expand : array('cont' => self::$request->content));
+			$form_name = self::form_name($cname, $this->id, $expand ?: array('cont' => self::$request->content));
 			if (!empty($this->attrs['path']))
 			{
 				$path = self::expand_name($this->attrs['path'], $expand['c'] ?? null, $expand['row'], $expand['c_'] ?? null, $expand['row_'] ?? null, $expand['cont']);
@@ -286,21 +286,21 @@ class Vfs extends File
 	{
 		$name = $_REQUEST['widget_id'];
 
-		// Find real path
+		// Find real path, could be "$app:$id:$path"
-		if($path[0] != '/')
+		if($path[0] !== '/')
 		{
 			$path = self::get_vfs_path($path);
 		}
 		$filename = $file['name'];
-		if ($path && substr($path,-1) != '/')
+		if ($path && substr($path,-1) !== '/')
 		{
-			// add extension to path
+			$parts = explode('.', $filename);
-			$parts = explode('.',$filename);
+			// check if path already contains a valid extension --> don't add another one
-			// check if path already contains a valid extension --> dont add an other one
+			$path_parts = explode('.', Api\Vfs::basename($path));
-			$path_parts = explode('.', $path);
+			if ((!($path_ext = array_pop($path_parts)) || Api\MimeMagic::ext2mime($path_ext) === 'application/octet-stream') &&
-			if (count($path_parts) > 2 && (!($extension = array_pop($path_parts)) || !Api\MimeMagic::ext2mime($extension)) &&
+				($extension = array_pop($parts) ?: Api\MimeMagic::mime2ext($file['mime'])))
-				($extension = array_pop($parts)) && Api\MimeMagic::ext2mime($extension))       // really an extension --> add it to path
 			{
+				// add extension to path
 				$path .= '.'.$extension;
 			}
 			$file['name'] = Api\Vfs::basename($path);

api/src/Storage/Customfields.php

@@ -637,7 +637,9 @@ class Customfields implements \IteratorAggregate
 
 	protected static function handle_file($entry_id, $field, $value)
 	{
-		$path = Api\Etemplate\Widget\Vfs::get_vfs_path($field['app'].":$entry_id:".$field['label']);
+		$path = Api\Etemplate\Widget\Vfs::get_vfs_path($field['app'].':'.$entry_id.':'.
+			(preg_match('/^[a-z_]+:[^:]+:(.+)$/', $field['name'], $matches) ? $matches[1] : $field['name']));
+
 		if($path)
 		{
 			foreach($value as $file)