one-off idempotent-ID computation is obsolete with newer DownloadableItem work
jzacsh opened this issue · 0 comments
jzacsh commented
need to continue the cleanup in #1079 by deleting IdempotentImportExecutorHelper and converting its usages to ImportableItem#getIdempotentId.
essentially we're switching now from having one-off implementations of ID building to an interface (I'm filing this because it's actually important to me as a open-closed-principle issue, since I'm extending via a new Media model and don't want to keep modifying the IdempotentImportExecutorHelper class).