Changeset 8720

Timestamp:

05/02/2019 07:51:30 AM (7 years ago)

Author:

coffee2code

Message:

Markdown Importer: Add automatic handling to recognize changes to parenting of manifest items.

Can adapt to newly added, removed, or changed parent.

See #4388.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-markdown/inc/class-importer.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-markdown/inc/class-importer.php

@@ -372 +372 @@
                         'post_excerpt' => sanitize_text_field( wp_slash( $excerpt ) ),
                 );
+
+                $markdown_entry = get_post_meta( $post_id, $this->manifest_entry_meta_key, true );
+
                 // If no title was extracted from markdown doc, use the value defined in manifest.
                 if ( is_null( $title ) ) {
-                        $markdown_entry = get_post_meta( $post_id, $this->manifest_entry_meta_key, true );
                         if ( ! empty( $markdown_entry['title'] ) ) {
                                 $post_data['post_title'] = sanitize_text_field( wp_slash( $markdown_entry['title'] ) );
@@ -380 +382 @@
                 } else {
                         $post_data['post_title'] = sanitize_text_field( wp_slash( $title ) );
+                }
+
+                $parent_id = wp_get_post_parent_id( $post_id );
+
+                if ( ! $markdown_entry ) {
+                        // Do nothing with regards to possibly changing post parent as we know
+                        // nothing about previous import.
+                }
+                // If post had a parent...
+                elseif ( $parent_id ) {
+                        $parent = $parent_id ? get_post( $parent_id ) : null;
+                        // ...but no parent is now defined, unset parent.
+                        if ( empty( $markdown_entry['parent'] ) ) {
+                                $post_data['post_parent'] = '';
+                        }
+                        // ...and it appears to differ from parent now defined, find new parent.
+                        elseif ( $markdown_entry['parent'] !== $parent->post_name ) {
+                                $find_parent = get_page_by_path( $markdown_entry['parent'], OBJECT, $this->get_post_type() );
+                                if ( $find_parent ) {
+                                        $post_data['post_parent'] = $find_parent->ID;
+                                }
+                        }
+                }
+                // Does this parentless post now have one newly defined?
+                elseif ( ! empty( $markdown_entry['parent'] ) ) {
+                        $find_parent = get_page_by_path( $markdown_entry['parent'], OBJECT, $this->get_post_type() );
+                        if ( $find_parent ) {
+                                $post_data['post_parent'] = $find_parent->ID;
+                        }
                 }