Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix Magento\Framework\DB\Adapter\Pdo\Mysql::getCreateTable() #562

Closed
wants to merge 1 commit into from
Closed

Bugfix Magento\Framework\DB\Adapter\Pdo\Mysql::getCreateTable() #562

wants to merge 1 commit into from

Conversation

SchumacherFM
Copy link
Member

This bugfix inserts the schema name into the SHOW CREATE TABLE statement for method getCreateTable().

Seems like it was previously forgotten ...

This bug is also in all Magento 1 versions.

@verklov verklov self-assigned this Apr 28, 2014
@verklov
Copy link
Contributor

verklov commented Apr 28, 2014

@SchumacherFM, thank you for your contribution! I created ticket in the backlog and will ask the team to verify the code you submitted. We will get back to you once the review is over.

magento-team added a commit that referenced this pull request Jun 16, 2014
* Added support for MTF Reporting Tool
* Framework improvements:
  * Covered the following Magento application components with unit tests:
      * `ConfigurableProduct/Helper/Data.php`
      * `ConfigurableProduct/Model/Export/RowCustomizer.php`
      * `ConfigurableProduct/Model/Product/Type/Configurable.php`
      * `ConfigurableProduct/Model/Product/Type/Plugin.php`
      * `ConfigurableProduct/Model/Quote/Item/QuantityValidator/Initializer/Option/Plugin/ConfigurableProduct.php`
      * `CatalogSearch/Helper/Data.php`
  * Covered Magento lib with unit tests:
      * `lib/internal/Magento/Framework/DB/Helper/AbstractHelper.php`
      * `lib/internal/Magento/Framework/DB/Tree/Node.php`
* Created Service API for Magento_Catalog Module:
  * Implemented the Product API
  * Implemented the ProductAttributeRead API
* Fixed bugs:
  * Fixed issues with form elements visibility on the backend
  * Fixed an issue where backend forms contained an excessive container
  * Fixed an issue where a wrong category structure was displayed on the Category page
  * Fixed an issue where the pub/index.php entry point was broken because of the obsolete constants
  * Fixed an issue where it was impossible to pass an empty array as an argument in DI configuration and layout updates
  * Fixed an issue with status and visibility settings of a related product on the backend
  * Fixed an issue with unused DB indexes, which used resources, but did not contribute to higher performance
  * Fixed an issue where it was possible to create a downloadable product without specifying a link or a file
  * Fixed an issue where a fatal error occured when opening a fixed bundle product with custom options page on the frontend
  * Fixed an issue where the was a wrong config key for backend cataloginventory
* Processed GitHub requests:
  * [#548] (#548) -- Console installer doesn't checks filesystem permissions
  * [#552] (#552) -- backend notifications sitebuild bug
  * [#562] (#562) -- Bugfix Magento\Framework\DB\Adapter\Pdo\Mysql::getCreateTable()
  * [#565] (#565) -- Magento\CatalogSearch\Model\Query::getResultCollection() not working
  * [#557] (#557) -- translation anomalies backend login page
* Added the following functional tests:
  * Advanced Search
  * Existing Customer Creation
  * Product Attribute Creation
  * Product Rating Creation
  * Sales Rule Creation
  * System Product Attribute Deletion
  * Tax Rate Creation
  * Tax Rule Deletion
  * Update Category
  * Update Category Url Rewrite
  * Update Product Url Rewrite
@verklov
Copy link
Contributor

verklov commented Jun 26, 2014

@SchumacherFM, the team has processed your contribution and deployed the code in version dev82. Thank you and looking forward for your new issues and pull requests!

@verklov verklov closed this Jun 26, 2014
magento-team pushed a commit that referenced this pull request Sep 2, 2015
magento-team pushed a commit that referenced this pull request Apr 26, 2016
okorshenko pushed a commit that referenced this pull request Dec 14, 2016
Fixed issue:
 - MAGETWO-59124 [Backport] - [GITHUB] Credit Card capture not associated with the Authorization since upgrade to 2.1.1 #6716 - for 2.1
 - MAGETWO-59578 Merge PayPal Ask MAGETWO-40517 and MAGETWO-52475 to 2.1.x
 - MAGETWO-59581 Merge PayPal Ask MAGETWO-53942 to 2.1.x
 - MAGETWO-59894 [Backport] Email Associated with PayPal Merchant Account Adaptation for On-Boarding
magento-engcom-team added a commit that referenced this pull request Apr 9, 2019
 - Merge Pull Request magento/graphql-ce#562 from magento/graphql-ce:graphQl-512-address.street-validation
 - Merged commits:
   1. 2d0eecc
   2. 27f7725
   3. c11b10b
   4. 722afe3
magento-devops-reposync-svc pushed a commit that referenced this pull request Dec 4, 2023
ACPT-1551: Modularity of ReloadProcessor::reloadApplicationState
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.

2 participants