#21 - D.T Migration Plugin - Activity Export & Import

[kodinkat]

• fixes: Activity export and import #21

---

--- --- --- --- --- --- --- --- ---

[corsacca]

@kodinkat

Default: yes

The created at on is wrong:

AI Review:

1. hist_ip falsely defaults to 127.0.0.1 — non-blocking but wrong

includes/class-dt-migration-export-file.php:410 and class-dt-migration-import-engine.php:529:
'hist_ip' => isset( $row['hist_ip'] ) ? (string) $row['hist_ip'] : '127.0.0.1',
Falsifying a historical IP is a bad default. '' or null (if the column is nullable) would be more
honest. Same nit for 'user_caps' ?? 'guest'.

3. Per-row $wpdb->insert() will be slow on large activity logs

class-dt-migration-import-engine.php:519 inserts one row at a time. The DT activity log can easily
have 10–100× the row count of records. On a 50k-record site, this could mean hundreds of thousands
of single-row inserts per batch. The existing import_post_user_meta_for_posts does the same — so
this PR isn't a regression — but it's the most likely perf cliff users will hit with the flag on.
Consider a multi-row INSERT (chunked, e.g., 500 rows/statement) as a follow-up. Worth at least a
comment.

Minor

7. Loop body in class-dt-migration-tab-import.php:196-199 does a count it doesn't use

$activity_n = 0;
if ( $this->import_preview_export_include_activity_log && isset( $activity_raw[ $post_type ] ) &&
is_array( $activity_raw[ $post_type ] ) ) {
$activity_n = count( $activity_raw[ $post_type ] );
}
$row = [ 'count' => is_array( $recs ) ? count( $recs ) : 0 ];
if ( $this->import_preview_export_include_activity_log ) {
$row['activity_log_count'] = $activity_n;
}
When import_preview_export_include_activity_log is false, $activity_n is computed (well, stays 0)
but never used. Reorder so the count() only runs inside the outer guard — small thing.

[kodinkat]

@corsacca review ready....