--- parser3/src/classes/json.C	2024/10/02 21:24:41	1.63
+++ parser3/src/classes/json.C	2025/10/05 19:41:27	1.68
@@ -1,7 +1,7 @@
 /** @file
 	Parser: @b json parser class.
 
-	Copyright (c) 2000-2023 Art. Lebedev Studio (http://www.artlebedev.com)
+	Copyright (c) 2000-2024 Art. Lebedev Studio (http://www.artlebedev.com)
 	Authors: Konstantin Morshnev <moko@design.ru>
 */
 
@@ -20,19 +20,24 @@
 #include "pa_vxdoc.h"
 #endif
 
-volatile const char * IDENT_JSON_C="$Id: json.C,v 1.63 2024/10/02 21:24:41 moko Exp $";
+volatile const char * IDENT_JSON_C="$Id: json.C,v 1.68 2025/10/05 19:41:27 moko Exp $";
 
 // class
 
 class MJson: public Methoded {
 public:
 	MJson();
+
+	override Value* get_element(const String&);
+	override const VJunction* put_element(const String&, Value*);
 };
 
 // global variable
 
 DECLARE_CLASS_VAR(json, new MJson);
 
+bool handle_array_default=true;
+
 // methods
 struct Json : public PA_Allocated {
 	Stack<VHashBase*> stack;
@@ -55,7 +60,7 @@ struct Json : public PA_Allocated {
 
 	Json(Charset* acharset): stack(), key_stack(), key(NULL), result(NULL), hook_object(NULL), hook_array(NULL), 
 		request(NULL), charset(acharset), taint(String::L_TAINTED), handle_double(true), handle_int(true), 
-		handle_array(true), distinct(D_EXCEPTION){}
+		handle_array(handle_array_default), distinct(D_EXCEPTION){}
 
 	bool set_distinct(const String &value){
 		if (value == "first") distinct = D_FIRST;
@@ -73,6 +78,31 @@ struct Json : public PA_Allocated {
 	}
 };
 
+
+Value* MJson::get_element(const String& aname) {
+	if (aname=="array"){
+		return new VString(*new String(handle_array_default ? "array" : "hash"));
+	}
+	return Methoded::get_element(aname);
+}
+
+const VJunction* MJson::put_element(const String& aname, Value* avalue) {
+	if (aname=="array"){
+		Json json(NULL);
+		if (avalue->get_string()){
+			const String& sarray=avalue->as_string();
+			if (json.set_handle_array(sarray)){
+				handle_array_default=json.handle_array;
+				return 0;
+			}
+			throw Exception(PARSER_RUNTIME, &sarray, "$json:array must be 'array' or 'hash'");
+		}
+		throw Exception(PARSER_RUNTIME, 0, "$json:array must be 'array' or 'hash'");
+	}
+	return Methoded::put_element(aname, avalue);
+}
+
+
 static void set_json_value(Json *json, Value *value){
 	VHashBase *top = json->stack.top_value();
 	if(json->key == NULL){
@@ -492,7 +522,7 @@ const String* Json_options::array_json_s
 				result << indent;
 				delim = get_array_delim(json_string_recursion);
 			}
-			result << value_json_string(i.key(), i.value() ? *i.value() : *VVoid::get(), *this);
+			result << value_json_string(String::Body::uitoa(i.index()), i.value() ? *i.value() : static_cast<Value&>(*VVoid::get()), *this);
 		}
 		result << "\n" << (indent=get_indent(json_string_recursion-1)) << "]";
 
@@ -501,7 +531,7 @@ const String* Json_options::array_json_s
 		bool need_delim=false;
 		for(ArrayValue::Iterator i(*array); i; i.next() ){
 			if(need_delim) result << ",\n";
-			result << value_json_string(i.key(), i.value() ? *i.value() : *VVoid::get(), *this);
+			result << value_json_string(String::Body::uitoa(i.index()), i.value() ? *i.value() : static_cast<Value&>(*VVoid::get()), *this);
 			need_delim=true;
 		}
 		result << "\n]";
@@ -531,7 +561,7 @@ const String* Json_options::array_compac
 					result << indent;
 					delim = get_array_delim(json_string_recursion);
 				}
-				result << value_json_string(i.key(), *i.value(), *this);
+				result << value_json_string(String::Body::uitoa(i.index()), *i.value(), *this);
 			}
 		}
 		result << "\n" << (indent=get_indent(json_string_recursion-1)) << "]";
@@ -542,7 +572,7 @@ const String* Json_options::array_compac
 		for(ArrayValue::Iterator i(*array); i; i.next() ){
 			if (i.value()){
 				if(need_delim) result << ",\n";
-				result << value_json_string(i.key(), *i.value(), *this);
+				result << value_json_string(String::Body::uitoa(i.index()), *i.value(), *this);
 				need_delim=true;
 			}
 		}
@@ -565,9 +595,10 @@ const String& value_json_string(String::
 			options.methods->put(v.type(), method ? method : VVoid::get());
 		}
 		if(method && !method->is_void()) {
+			static const String::Body sindent("indent");
 			Junction* junction=method->get_junction();
 			HashStringValue* params_hash=options.params && options.indent ? options.params->get_hash() : NULL;
-			Temp_hash_value<HashStringValue, Value*> indent(params_hash, "indent", new VString(*new String(options.indent, String::L_AS_IS)));
+			Temp_hash_value<HashStringValue, Value*> indent(params_hash, sindent, new VString(*new String(options.indent, String::L_AS_IS)));
 
 			Value *params[]={new VString(*new String(key, String::L_JSON)), &v, options.params ? options.params : VVoid::get()};
 
@@ -590,65 +621,52 @@ static void _string(Request& r, MethodPa
 		if(HashStringValue* options=params.as_hash(1)) {
 			json.params=&params[1];
 			HashStringValue* methods=new HashStringValue();
-			int valid_options=0;
 			HashStringValue* vvalue;
 			for(HashStringValue::Iterator i(*options); i; i.next() ){
 				String::Body key=i.key();
 				Value* value=i.value();
 				if(key == "skip-unknown"){
 					json.skip_unknown=r.process(*value).as_bool();
-					valid_options++;
 				} else if(key == "one-line"){
 					json.one_line=r.process(*value).as_bool();
-					valid_options++;
 				} else if(key == "date" && value->is_string()){
 					const String& svalue=value->as_string();
 					if(!json.set_date_format(svalue))
 						throw Exception(PARSER_RUNTIME, &svalue, "must be 'sql-string', 'gmt-string', 'iso-string' or 'unix-timestamp'");
-					valid_options++;
 				} else if(key == "indent"){
 					if(value->is_string()){
 						json.indent=value->as_string().cstr();
 						json.json_string_recursion=strlen(json.indent);
 					} else json.indent=r.process(*value).as_bool() ? "" : NULL;
-					valid_options++;
 				} else if(key == "table" && value->is_string()){
 					const String& svalue=value->as_string();
 					if(!json.set_table_format(svalue))
 						throw Exception(PARSER_RUNTIME, &svalue, "must be 'array', 'object' or 'compact'");
-					valid_options++;
 				} else if(key == "array" && value->is_string()){
 					const String& svalue=value->as_string();
 					if(!json.set_array_format(svalue))
 						throw Exception(PARSER_RUNTIME, &svalue, "must be 'array', 'object' or 'compact'");
-					valid_options++;
 				} else if(key == "file" && value->is_string()){
 					const String& svalue=value->as_string();
 					if(!json.set_file_format(svalue))
 						throw Exception(PARSER_RUNTIME, &svalue, "must be 'base64', 'text' or 'stat'");
-					valid_options++;
 				} else if(key == "void" && value->is_string()){
 					const String& svalue=value->as_string();
 					if(!json.set_void_format(svalue))
 						throw Exception(PARSER_RUNTIME, &svalue, "must be 'string' or 'null'");
-					valid_options++;
 #ifdef XML
 				} else if(key == "xdoc" && (vvalue = value->get_hash())){
 					json.xdoc_options=new XDocOutputOptions();
 					json.xdoc_options->append(r, vvalue);
-					valid_options++;
 #endif
 				} else if(Junction* junction=value->get_junction()){
 					if(!junction->method || !junction->method->params_names || junction->method->params_count != 3)
 						throw Exception(PARSER_RUNTIME, 0, "$.%s must be parser method with 3 parameters", key.cstr());
 					methods->put(key, value);
-					valid_options++;
-				}
+				} else
+					throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
 			}
 
-			if(valid_options!=options->count())
-				throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
-
 			// special handling for $._default 
 			if(VHashBase* vhash=dynamic_cast<VHashBase*>(&params[1]))
 				if(Value* value=vhash->get_default()) {
@@ -671,7 +689,7 @@ static void _string(Request& r, MethodPa
 		for(char *c=result;*c;c++)
 			if(*c=='\n')
 				*c=' ';
-		result_body=result;
+		result_body=String::Body(result);
 	}
 	r.write(*new String(result_body, String::L_AS_IS));
 }