Currently the limit of items to process per batch iteration is hardcoded in EntityImporterBatchProcess. Using a getter instead of accessing the limit variable directly would allow source plugins to manage that value.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

justin2pin created an issue. See original summary.

justin2pin’s picture

Status: Needs work » Needs review
FileSize
1.9 KB

Patch attached with getter for limit count.

droath’s picture

Made some minor adjustments, so that we don't have to call a get method which sets the default limit. Then after thinking about it, it made sense to me to just remove the setLimitCount(), and only provide a getLimitCount(). Then the source plugin that's extending the EntityImportSourceLimitIteratorBase class has all the control of what the limit count should be, or it can be set via the UI.

  • droath committed e0631a5 on 8.x-1.x authored by justin2pin
    Issue #3088838 by justin2pin, droath: Allow source plugins to set their...
droath’s picture

Status: Needs review » Fixed
droath’s picture

Status: Fixed » Closed (fixed)