Skip to content

Conversation

@saurabh-parekh
Copy link
Owner

Original Pull Request

magento#15891

Remove parameter from method calling

Description

Method definition does not require any parameters. But when it was actually called extra parameter was passed. I remove extra parameter while calling method.

Fixed Issues (if relevant)

N/A

Manual testing scenarios

1 You can check at magento admin panel -> Marketing -> SEO & Search -> URL Rewrites -> Add/Edit with product.

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

Alex Paliarush and others added 30 commits May 30, 2018 19:31
…to-graphql-ce-48

[architects] MAGETWO-92260: GraphQL - Fix urlResolver query to support relative path magento#13
…ustom source model in the adminhtml doesn't render selected value
[performance] MAGETWO-92306: [Forwardport] Fix Category Product Indexer
…ustom source model in the adminhtml doesn't render selected value
 - merged latest code from mainline branch
Stanislav Idolov and others added 28 commits June 6, 2018 12:40
Accepted Public Pull Requests:
 - magento#15853: Fixed return type hinting in DocBlocks for Wishlist module (by @rogyar)
 - magento#15695: Fixed a couple of typos  (by @dverkade)
 - magento#15833: [Forwardport] Format the javascript code magento#10 (by @chirag-wagento)
 - magento#15755: [Forwardport] Error 500 in Module Manager (by @vijay-wagento)
 - magento#15794: [Forwardport] [Resolved : Styling <select> by changing less variables in Luma theme� (by @hitesh-wagento)
 - magento#15809: [Forwardport] fix: support multiple minisearch widget instances (by @DanielRuf)
 - magento#15558: Fixes in catalog component blocks [2.3-develop] (by @mhauri)
 - magento#15572: [Forwardport] Docblock typo fixes (by @rogyar)
 - magento#15748: [Forwardport] Prevent multiple add-to-cart initializations in case of ajax loaded product listing (by @vijay-wagento)
 - magento#15745: [Forwardport] Fix method name (typo) (by @vijay-wagento)
 - magento#15746: [Forwardport] Resolved : UI Component listing action column outside of screen when... (by @vgelani)
 - magento#15692: Fix minor issues in ui export converter classes (by @dmytro-ch)
 - magento#15649: [Forwardport] Fixes in config module (by @mhauri)
 - magento#15540: fix for Wrong price amount on product page magento#11717 (by @HirokazuNishi)


Fixed GitHub Issues:
 - magento#15192: Module Manager module grid is not working Magento 2.2.4 (reported by @jignesh-baldha) has been fixed in magento#15755 by @vijay-wagento in 2.3-develop branch
   Related commits:
     1. 486f7a7
     2. 1b5d1d1

 - magento#15608: Styling <select> by changing less variables in Luma theme doesn't work as expected (reported by @denistrator) has been fixed in magento#15794 by @hitesh-wagento in 2.3-develop branch
   Related commits:
     1. 9fa6425

 - magento#14153: UI Component listing action column outside of screen when too many columns (reported by @milansimek) has been fixed in magento#15746 by @vgelani in 2.3-develop branch
   Related commits:
     1. 365bfb4

 - magento#11717: Wrong price amount on product page (reported by @HirokazuNishi) has been fixed in magento#15540 by @HirokazuNishi in 2.3-develop branch
   Related commits:
     1. cf05a84
     2. 67a2038
…he cache issue for currencies with no symbol
[TSG] Upporting for 2.3 (pr21) (2.3.0)
…5921

 - Merge Pull Request magento#15921 from DanielRuf/magento2:2.3-develop-PR-port-15871
 - Merged commits:
   1. 4d03776


 - Merge Pull Request magento#15912 from NamrataChangani/magento2:2.3-develop-PR-port-15825
 - Merged commits:
   1. 3782793
…ets. magento#15911

 - Merge Pull Request magento#15911 from NamrataChangani/magento2:2.3-develop-PR-port-15795
 - Merged commits:
   1. fdaf7cb
…e currencies with no symbol magento#15903

 - Merge Pull Request magento#15903 from dmytro-ch/magento2:fix/2.3-cache-issue-for-currencies-with-no-symbol
 - Merged commits:
   1. 23d76ed
Accepted Public Pull Requests:
 - magento#15903: [Forwardport] Complete the fix for cache issue due to the currencies with no symbol (by @dmytro-ch)
 - magento#15911: [Forwardport] Remove double semicolon from the style sheets. (by @namratachangani)
 - magento#15912: [Forwardport] Fixed set template syntax issue (by @namratachangani)
 - magento#15921: [Forwardport] chore: remove unused less import (by @DanielRuf)
 - magento#15867: [Forwardport] Update webapi.xml to fix typo (by @dmytro-ch)
 - magento#15847: [Up-port] check if order data is available to incl ec (by @torhoehn)


Fixed GitHub Issues:
 - magento#12221: Google analytics pageview being triggered twice (reported by @alexhadley) has been fixed in magento#15847 by @torhoehn in 2.3-develop branch
   Related commits:
     1. e46e5a5
…437-Category-Product-Indexer

[honey] MAGETWO-91437: Catalog_category_product_index table doesn't update after deleting category
saurabh-parekh pushed a commit that referenced this pull request Jun 8, 2018
[Backport] Added language translation for message string
@saurabh-parekh saurabh-parekh merged commit fe2ad82 into 2.3-develop Jun 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.