get method order fix

Author: akos-tk

api/openapi.json

@@ -6691,7 +6691,7 @@
   },
   "/v2/accounts/{account_id}/events/emulate": {
    "post": {
-    "description": "Emulate sending message to blockchain",
+    "description": "Emulate sending message to retrieve account-specific events",
     "operationId": "emulateMessageToAccountEvent",
     "parameters": [
      {
@@ -7546,16 +7546,6 @@
        },
        "type": "array"
       }
-     },
-     {
-      "in": "query",
-      "name": "fix_order",
-      "required": false,
-      "schema": {
-       "default": true,
-       "description": "A temporary fix to switch to a scheme with direct ordering of arguments. \nIf equal to false, then the method takes arguments in direct order,\ne.g. for get_nft_content(int index, cell individual_content) we pass a list of arguments [index, individual_content].\nIf equal to true, then the method takes arguments in reverse order, e.g. [individual_content, index].",
-       "type": "boolean"
-      }
      }
     ],
     "responses": {
@@ -8181,7 +8171,7 @@
   },
   "/v2/events/emulate": {
    "post": {
-    "description": "Emulate sending message to blockchain",
+    "description": "Emulate sending message to retrieve general blockchain events",
     "operationId": "emulateMessageToEvent",
     "parameters": [
      {
@@ -10720,7 +10710,7 @@
   },
   "/v2/traces/emulate": {
    "post": {
-    "description": "Emulate sending message to blockchain",
+    "description": "Emulate sending message to retrieve with a detailed execution trace",
     "operationId": "emulateMessageToTrace",
     "parameters": [
      {
@@ -10823,7 +10813,7 @@
   },
   "/v2/wallet/emulate": {
    "post": {
-    "description": "Emulate sending message to blockchain",
+    "description": "Emulate sending message to retrieve the resulting wallet state",
     "operationId": "emulateMessageToWallet",
     "parameters": [
      {

api/openapi.yml

@@ -428,17 +428,6 @@ paths:
             items:
               type: string
             example: [ "0:9a33970f617bcd71acf2cd28357c067aa31859c02820d8f01d74c88063a8f4d8" ]
-        - name: fix_order
-          in: query
-          required: false
-          schema:
-            type: boolean
-            description: |-
-              A temporary fix to switch to a scheme with direct ordering of arguments. 
-              If equal to false, then the method takes arguments in direct order,
-              e.g. for get_nft_content(int index, cell individual_content) we pass a list of arguments [index, individual_content].
-              If equal to true, then the method takes arguments in reverse order, e.g. [individual_content, index].
-            default: true
       responses:
         '200':
           description: method execution result

pkg/api/account_handlers.go

@@ -17,12 +17,10 @@ import (
 	"go.uber.org/zap"
 	"golang.org/x/exp/maps"
 
-	"github.com/tonkeeper/opentonapi/pkg/addressbook"
-	"golang.org/x/exp/slices"
-
 	"github.com/cespare/xxhash/v2"
 	"github.com/go-faster/jx"
 	"github.com/tonkeeper/opentonapi/internal/g"
+	"github.com/tonkeeper/opentonapi/pkg/addressbook"
 	"github.com/tonkeeper/opentonapi/pkg/core"
 	"github.com/tonkeeper/opentonapi/pkg/oas"
 	"github.com/tonkeeper/tongo"
@@ -210,10 +208,6 @@ func (h *Handler) ExecGetMethodForBlockchainAccount(ctx context.Context, params
 		}
 		return nil, toError(http.StatusInternalServerError, err)
 	}
-	// TODO: remove parameter after user migration
-	if params.FixOrder.IsSet() && params.FixOrder.Value == true && len(params.Args) > 1 {
-		slices.Reverse(params.Args)
-	}
 	key, err := getMethodCacheKey(account.ID, params.MethodName, contract.LastTransactionLt, params.Args)
 	if err != nil {
 		return nil, toError(http.StatusInternalServerError, err)