Set comment_post_ID to 0 if not set; revert message check change in import-csv.

Author: gitlost

features/user-import-csv.feature

@@ -52,9 +52,10 @@ Feature: Import users from CSV
       """
 
     When I try `wp user import-csv user-invalid.csv`
-    Then STDERR should be:
+	# Message changed from "Only lowercase..." to "Usernames can contain only lowercase..." in `wpmu_validate_user_signup()` WP 4.4 https://core.trac.wordpress.org/ticket/33336
+    Then STDERR should contain:
       """
-      Warning: Usernames can only contain lowercase letters (a-z) and numbers.
+      lowercase letters (a-z) and numbers
       """
     And the return code should be 0
 

src/Comment_Command.php

@@ -69,6 +69,9 @@ public function create( $args, $assoc_args ) {
 				if ( !$post ) {
 					return new WP_Error( 'no_post', "Can't find post $post_id." );
 				}
+			} else {
+				// Make sure it's set for older WP versions else get undefined PHP notice.
+				$params['comment_post_ID'] = 0;
 			}
 
 			// We use wp_insert_comment() instead of wp_new_comment() to stay at a low level and
@@ -163,7 +166,9 @@ public function generate( $args, $assoc_args ) {
 			$notify = \WP_CLI\Utils\make_progress_bar( 'Generating comments', $assoc_args['count'] );
 		}
 
-		$comment_post_ID = empty( $assoc_args['post_id'] ) ? 1 : $assoc_args['post_id'];
+		// Make sure comment_post_ID set for older WP versions else get undefined PHP notice.
+		$comment_post_ID = isset( $assoc_args['post_id'] ) ? $assoc_args['post_id'] : 0;
+
 		$comment_count = wp_count_comments();
 		$total = (int )$comment_count->total_comments;
 		$limit = $total + $assoc_args['count'];