adjust code review

mercadopago · Aug 22, 2023 · a0150f1 · a0150f1
1 parent a5d93bd
commit a0150f1
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/src/MercadoPago/Client/Payment/PaymentClient.php b/src/MercadoPago/Client/Payment/PaymentClient.php
@@ -106,14 +106,14 @@ public function capture(int $id, ?float $amount, ?MPRequestOptions $request_opti
     /**
      *  Method responsible for search payments.
      * @param \MercadoPago\Net\MPSearchRequest $request search request.
-     * @param mixed $requestOptions request options to be sent.
+     * @param mixed $request_options request options to be sent.
      * @return \MercadoPago\Resources\PaymentSearch search results.
      */
-    public function search(MPSearchRequest $request, ?MPRequestOptions $requestOptions = null): PaymentSearch
+    public function search(MPSearchRequest $request, ?MPRequestOptions $request_options = null): PaymentSearch
     {
         try {
-            $queryParams = isset($request) ? $request->getParameters() : null;
-            $response = parent::send(self::$URL_SEARCH, HttpMethod::GET, null, $queryParams, $requestOptions);
+            $query_params = isset($request) ? $request->getParameters() : null;
+            $response = parent::send(self::$URL_SEARCH, HttpMethod::GET, null, $query_params, $request_options);
             $result = Serializer::deserializeFromJson(PaymentSearch::class, $response->getContent());
             $result->setResponse($response);
             return $result;

diff --git a/src/MercadoPago/Client/Preference/PreferenceClient.php b/src/MercadoPago/Client/Preference/PreferenceClient.php
@@ -85,14 +85,14 @@ public function update(string $id, array $request, ?MPRequestOptions $request_op
     /**
      *  Method responsible for search preferences.
      * @param \MercadoPago\Net\MPSearchRequest $request search request.
-     * @param mixed $requestOptions request options to be sent.
+     * @param mixed $request_options request options to be sent.
      * @return \MercadoPago\Resources\PreferenceSearch search results.
      */
-    public function search(MPSearchRequest $request, ?MPRequestOptions $requestOptions = null): PreferenceSearch
+    public function search(MPSearchRequest $request, ?MPRequestOptions $request_options = null): PreferenceSearch
     {
         try {
-            $queryParams = isset($request) ? $request->getParameters() : null;
-            $response = parent::send(self::$URL_SEARCH, HttpMethod::GET, null, $queryParams, $requestOptions);
+            $query_params = isset($request) ? $request->getParameters() : null;
+            $response = parent::send(self::$URL_SEARCH, HttpMethod::GET, null, $query_params, $request_options);
             $result = Serializer::deserializeFromJson(PreferenceSearch::class, $response->getContent());
             $result->setResponse($response);
             return $result;

diff --git a/src/MercadoPago/Resources/Preference/Item.php b/src/MercadoPago/Resources/Preference/Item.php
@@ -24,7 +24,7 @@ class Item
     public ?int $quantity;
 
     /** Unit price. */
-    public ?float $unitPrice;
+    public ?float $unit_price;
 
     /** Currency ID. ISO_4217 code. */
     public ?string $currency_id;