Description: ------------ Removing multiple subfields using deleteSubfield does not work as expected. The problem is that the method doesn't call $n->setPosition($pos) on the subfields that follow the removed field. I wasn't familiar with the SplDoublyLinkedList structure from before, but I tried cooking together a patch that at least solves my issue. Test script: --------------- require 'vendor/autoload.php'; require 'File/MARC/Data_Field.php'; $field = new File_MARC_Data_Field(650, [ new File_MARC_Subfield('9', 'test1'), new File_MARC_Subfield('9', 'test2'), new File_MARC_Subfield('0', 'test3'), new File_MARC_Subfield('9', 'test4'), ]); echo "--- Before: ---\n$field\n\n"; foreach ($field->getSubfields('9') as $subfield) { echo "Deleting subfield: $subfield\n"; $field->deleteSubfield($subfield); } echo "\n--- After: ---\n$field\n\n"; Expected result: ---------------- --- Before: --- 650 _9test1 _9test2 _0test3 _9test4 Deleting subfield: [9]: test1 Deleting subfield: [9]: test2 Deleting subfield: [9]: test4 --- After: --- 650 _0test3 Actual result: -------------- --- Before: --- 650 _9test1 _9test2 _0test3 _9test4 Deleting subfield: [9]: test1 Deleting subfield: [9]: test2 Deleting subfield: [9]: test4 --- After: --- 650 _9test2 _9test4

Added patch List.php.patch.

Would you be willing to add a test script that shows the call not working until your patch is applied?

Wait - I'll just adapt what you've put here into a test script. Thank you!