Merge pull request #47 from Caprico85/master

Use collation safe comparison for config and attribute value cleanup

Author: sprankhub

src/FIREGENTO/Magento/Command/Eav/RemoveUnusedMediaCommand.php

@@ -97,7 +97,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
                     continue;
                 }
 
-                $value = $coreRead->fetchOne('SELECT value FROM ' . $mediaGallery . ' WHERE value = ?', array($filePath));
+                $value = $coreRead->fetchOne('SELECT value FROM ' . $mediaGallery . ' WHERE BINARY value = ?', array($filePath));
 
                 if ($value == false) {
                     $row = array();

src/FIREGENTO/Magento/Command/Eav/RestoreUseDefaultValueAttributesCommand.php

@@ -57,7 +57,7 @@ public function execute(InputInterface $input, OutputInterface $output)
                 foreach ($rows as $row) {
                     // Select the global value if it's the same as the non-global value
                     $results = $db->fetchAll('SELECT * FROM ' . $fullTableName
-                        . ' WHERE entity_type_id = ? AND attribute_id = ? AND store_id = ? AND entity_id = ? AND value = ?',
+                        . ' WHERE entity_type_id = ? AND attribute_id = ? AND store_id = ? AND entity_id = ? AND BINARY value = ?',
                         array($row['entity_type_id'], $row['attribute_id'], 0, $row['entity_id'], $row['value'])
                     );
 

src/FIREGENTO/Magento/Command/Eav/RestoreUseDefaultValueConfigCommand.php

@@ -50,11 +50,11 @@ public function execute(InputInterface $input, OutputInterface $output)
 
             $configData = $db->fetchAll('SELECT DISTINCT path, value FROM ' . $this->_prefixTable('core_config_data') . ' WHERE scope_id = 0');
             foreach($configData as $config) {
-                $count = $db->fetchOne('SELECT COUNT(*) FROM ' . $this->_prefixTable('core_config_data') .' WHERE path = ? AND value = ?', array($config['path'], $config['value']));
+                $count = $db->fetchOne('SELECT COUNT(*) FROM ' . $this->_prefixTable('core_config_data') .' WHERE path = ? AND BINARY value = ?', array($config['path'], $config['value']));
                 if($count > 1) {
                     $output->writeln('Config path ' . $config['path'] . ' with value ' . $config['value']. ' has ' . $count . ' values; deleting non-default values');
                     if(!$isDryRun) {
-                        $db->query('DELETE FROM ' . $this->_prefixTable('core_config_data') . ' WHERE path = ? AND value = ? AND scope_id != ?', array($config['path'], $config['value'], 0));
+                        $db->query('DELETE FROM ' . $this->_prefixTable('core_config_data') . ' WHERE path = ? AND BINARY value = ? AND scope_id != ?', array($config['path'], $config['value'], 0));
                     }
                     $removedConfigValues += ($count-1);
                 }