Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@i2h3
Copy link
Collaborator

@i2h3 i2h3 commented Sep 11, 2025

This is necessary to make the initializer available outside of the NextcloudKit module, in example in test implementations of NextcloudFileProviderKit.

This is necessary to make the initializer available outside of the NextcloudKit module, in example in test implementations of NextcloudFileProviderKit.

Signed-off-by: Iva Horn <[email protected]>
@i2h3 i2h3 self-assigned this Sep 11, 2025
@i2h3 i2h3 added the enhancement New feature request label Sep 11, 2025
@i2h3 i2h3 merged commit cc0fa30 into main Sep 11, 2025
2 of 4 checks passed
@i2h3 i2h3 deleted the nktrash-init branch September 11, 2025 13:35
public var trashbinOriginalLocation: String
public var trashbinDeletionTime: Date

public init(ocId: String = "", contentType: String = "", typeIdentifier: String = "", date: Date = Date(), directory: Bool = false, fileId: String = "", fileName: String = "", filePath: String = "", hasPreview: Bool = false, iconName: String = "", size: Int64 = 0, classFile: String = "", trashbinFileName: String = "", trashbinOriginalLocation: String = "", trashbinDeletionTime: Date = Date()) {
Copy link
Collaborator

@mpivchev mpivchev Sep 12, 2025

Choose a reason for hiding this comment

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

This needs to be increased, ill do it in my PR

Ask if possible to change this first

@mpivchev
Copy link
Collaborator

mpivchev commented Sep 12, 2025

Probably should not have been merged yet. There are obvious Lint errors

@mpivchev
Copy link
Collaborator

@i2h3 Is it possible to reduce the size of the init? Maybe by moving defaults to the properties themselves? Also, are all fields required to be params? If possible, we can exclude some and keep the default value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants