Merge pull request #93 from antigenius/null-export-fix

Corrects NULL export values.

Author: schlessera

features/search-replace-export.feature

@@ -359,3 +359,17 @@ Feature: Search / replace with file export
       """
       'test-remove-placeholder-escape{'
       """
+
+  Scenario: NULLs exported as NULL and not null string
+    Given a WP install
+    And I run `wp db query "INSERT INTO wp_postmeta VALUES (9999, 9999, NULL, 'foo')"`
+
+    When I run `wp search-replace bar replaced wp_postmeta --export`
+    Then STDOUT should contain:
+      """
+     ('9999', '9999', NULL, 'foo')
+      """
+    And STDOUT should not contain:
+      """
+     ('9999', '9999', '', 'foo')
+      """

src/Search_Replace_Command.php

@@ -558,8 +558,18 @@ private function write_sql_row_fields( $table, $rows ) {
 		$export_insert_size = $this->export_insert_size;
 
 		foreach($rows as $row_fields) {
-			$sql .= '(' . join( ', ', array_fill( 0, count( $row_fields ), '%s' ) ) . ')';
-			$values = array_merge( $values, array_values( $row_fields ) );
+			$subs = array();
+			
+			foreach( $row_fields as $field_value ) {
+				if ( null === $field_value ) {
+					$subs[] = 'NULL';
+				} else {
+					$subs[] = '%s';
+					$values[] = $field_value;
+				}
+			}
+			
+			$sql .= '(' . join( ', ', $subs ) . ')';
 
 			// Add new insert statement if needed. Before this we close the previous with semicolon and write statement to sql-file.
 			// "Statement break" is needed: