Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vladimir-m’s picture

vladimir-m’s picture

Updated Patch.

13rac1’s picture

Status: Needs review » Needs work

The patch applies cleanly and changes the UI, but I must be missing something. It keeps showing the same values for "Imported" and "Needs Update" I re-run the import and nothing changes. I have "track_changes" set, but I don't see how it would be an issue.

mikeryan’s picture

Right, this patch includes both STATUS_IMPORTED and STATUS_NEEDS_UPDATE in the "Needs update" count, which is confusing (and usually redundant).

If we're going to add more info to the dashboard, I think STATUS_IGNORED is the big thing we need - STATUS_NEEDS_UPDATE is usually ephemeral (most often it's only set for the duration of a --update), while ignored rows are the major explanation for persistent discrepancies between Items-Imported and Unprocessed.

And, if we're adding this info to the UI, we should also add it to the drush migrate-status command.

pifagor’s picture

Status: Needs work » Closed (outdated)