Uploaded image for project: 'PHP Driver: Library'
  1. PHP Driver: Library
  2. PHPLIB-1523

Remove iterators for database, collection, and index enumeration

XMLWordPrintableJSON

    • Type: Icon: Task Task
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • 2.0.0
    • Affects Version/s: None
    • Component/s: None
    • None
    • PHP Drivers
    • Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      API docs for the listDatabases, listCollections, and listIndexes methods need to be updated.

      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      Tutorials for database/collection/index enumeration may need updating. The usage is the same as before.

      3. Which versions of the driver/connector does this apply to?
      2.0+

      Show
      1. What would you like to communicate to the user about this feature? API docs for the listDatabases , listCollections , and listIndexes methods need to be updated. 2. Would you like the user to see examples of the syntax and/or executable code and its output? Tutorials for database/collection/index enumeration may need updating. The usage is the same as before. 3. Which versions of the driver/connector does this apply to? 2.0+

      The iterators for DatabaseInfo, CollectionInfo, and IndexInfo can be removed. Instead, create a CachingIterator that decorates a CallbackIterator:

      // First class callable syntax, needs new factory in info classes
return new CachingIterator(new CallbackIterator($cursor, IndexInfo::fromDatabaseResult(...)));


// Alternatively, create a static closure to instantiate the info object
return new CachingIterator(new CallbackIterator($cursor, static fn (array $info) => new IndexInfo($info)));

            Assignee:
            andreas.braun@mongodb.com Andreas Braun
            Reporter:
            andreas.braun@mongodb.com Andreas Braun
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: