Skip to content

align listContents with local filesystem adapter - #16

Merged
ylebre merged 1 commit into
mainfrom
feature/flysystem-3
Nov 3, 2026
Merged

ylebre merged 1 commit into
mainfrom
feature/flysystem-3

Conversation

@ylebre

@ylebre ylebre commented Oct 6, 2026

Copy link
Copy Markdown
Member

No description provided.

@Potherca Potherca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One question, across several files.

Comment thread src/Nextcloud.php
final public function getAttributes(string $path): FileAttributes
{
$node = $this->folder->get($path);
$metaData = $this->normalizeNodeInfo($node);

@Potherca Potherca Oct 13, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this also need the dir/file fix?

Comment thread src/NextcloudCalendar.php
$calendarId = $this->getCalendarId($path);
if ($calendarId !== null) {
$calendar = $this->calDavBackend->getCalendarById($calendarId);
$metaData = $this->normalizeCalendar($calendar);

@Potherca Potherca Oct 13, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this also need the dir/file fix?

Comment thread src/NextcloudContacts.php
$addressBookId = $this->getAddressBookId($path);
if ($addressBookId !== null) {
$addressBook = $this->cardDavBackend->getAddressBookById($addressBookId);
$metaData = $this->normalizeAddressBook($addressBook);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this also need the dir/file fix?

@ylebre
ylebre merged commit 1be7f7f into main Nov 3, 2026
14 checks passed
@ylebre
ylebre deleted the feature/flysystem-3 branch November 3, 2026 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants