Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature/pasteboard] Pasteboard Actions #514

Merged
merged 126 commits into from
Jul 22, 2021
Merged

Conversation

hosy
Copy link
Collaborator

@hosy hosy commented Oct 2, 2019

Description

This PR implements new actions, regarding the system and app pasteboard:

  • Copy
  • Paste (Import)
  • Cut

Inside the app it is possible to cut and copy items from one path to a new path.
Outside the app, it is possible to provide items to other apps and the system.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

QA

Test plan: https://github.com/owncloud/QA/blob/master/Mobile/iOS-app/Version%2011.6.1/Pasteboard%20options.md

Reports:

felix-schwarz and others added 30 commits June 30, 2019 22:57
- uploads use new OCCoreOptionAutomaticConflictResolutionNameStyle option to automatically resolve naming conflicts during upload
- Duplicate action uses the new OCCore name suggestion API to
	- determine naming conflicts and resolve them automatically
	- match the name style of the file to duplicate, f.ex.
		- duplicating "File copy.jpg" will create "File copy 2.jpg"
		- duplicating "File (1).jpg" will create "File (2).jpg"
		- duplicating "File Kopie 2.jpg" will create "File Kopie 3.jpg"
- folder creation uses the new OCCore name suggestion API to
	- detect naming conflicts beforehand
	- pre-fill the new folder name name with an unused name directly
…es will be deleted to give the user a UI feedback
…y log file using new SDK protocol

- Show SDK commit hash in Settings
* Rename updates the currently visible file in the gallery

* Keeping gallery alive fix

- Don’t pop to the root view controller on destructive action by default
- Only pop to the root if all items have been deleted
- Select next item upon deletion of the current ones
- Update UIPageViewController data source if number of items changes due e.g. to duplication or deletion

* Small fixes

* Fixed review findings
* Media player implemented using AVKit

* Added possibility to stream audio / video

* Improved error handling

* Fixed review finding

* Added background audio, airplay and PiP mode

* Propagating safe area to the main view of AVPlayerViewController

* Another fix for safe area considering layout of AVPlayerViewController

* Updated to latest develop version of SDK
Log device, version and locale info #446
Added an activity indicator for deleting offline copies #438
Naming improvements for upload, duplicate and folder creation #430
…was dismissed

- use the current development SDK
- fixed create folder action
- delete local copy, if needed
- code review changes
	- add additional safeguard so duplicate files are only deleted if they were actually duplicated (previously non-duplicate files could be removed if duplication or folder creation failed)
	- remove temporary container folder, too
- add 2 second delay before returning the core to give the core an opportunity to schedule the upload on a NSURLSession
@michaelstingl
Copy link
Contributor

(6)

Option Cut works only inside the oC app, is that expected?

„Cut“ to another storage should be an implicit copy I‘d say.

@jesmrec
Copy link
Contributor

jesmrec commented Jul 15, 2021

(6)

Option Cut works only inside the oC app, is that expected?

„Cut“ to another storage should be an implicit copy I‘d say.

i've slightly modified the definition of (6): Cut only works in the same oC account.

If cut content is pasted in another provider -> not pasted
If cut content is pasted in another oC account inside the app -> not pasted

- copying and cuting of multiple files is now possible
- pasting of multiple files is now working
@hosy
Copy link
Collaborator Author

hosy commented Jul 19, 2021

@jesmrec I fixed all QA findings. You can give it a new try.

  • (1) multiple items can be copied and pastes
  • (2) "Copy" notification was fixed and shows the number of total copies items
  • (3) copy a folder only works inside the oC app (folders cannot be pasted into Files.app or third party apps)
  • (4) the pasteboard does not preserve the file name
  • (5) pasting text (e.g. from Safari) results in more files (because the pasteboard provides different output formats. This may be solved in a further development step. But currently the user have different options, which format the user wants to keep)
  • (6) cut an item is only possible within the account

@jesmrec
Copy link
Contributor

jesmrec commented Jul 20, 2021

(1), (2) -> fixed
(3), (4) -> out of scope
(5) -> to improve in a separate issue?
(6) -> If Cut only works in the same account, is there any difference with the Move option?

@hosy
Copy link
Collaborator Author

hosy commented Jul 20, 2021

@jesmrec

(1), (2) -> fixed
(3), (4) -> out of scope
(5) -> to improve in a separate issue?

Yes, we should have a look how it works for us and which ideas we have to solve this problem. But yes, this should be improved in a separate issue.

(6) -> If Cut only works in the same account, is there any difference with the Move option?
You are right, it is like the Move option, but without the directory picker.

@hosy
Copy link
Collaborator Author

hosy commented Jul 20, 2021

@jesmrec I think I found a solution to preserve the file name. I will commit an update soon.

@jesmrec
Copy link
Contributor

jesmrec commented Jul 20, 2021

about (3)

(3) copy a folder only works inside the oC app (folders cannot be pasted into Files.app or third party apps)

Copying a folder from one oC account to another oC account inside the app. Should this work?

now, it doesn't

@jesmrec
Copy link
Contributor

jesmrec commented Jul 20, 2021

(7)

another question: is there any space/storage restriction in the clipboard? copying many files (or less bigger files), they are not all pasted.

@jesmrec
Copy link
Contributor

jesmrec commented Jul 20, 2021

After the 2nd round, pending stuff:

  • Checking if pasted items could preserve the original name

  • Folders (not) pasted to different account in the same app.

  • Question about the size of the copied content

Some improvements that could be addressed in separate issues, as new iterations:

  • Pasting text from clipboard: selecting format to paste the text (report (5) )

  • Add a spinner/waiter while the items are copied: if many items are selected to be copied to clipboard, some time runs till everything is copied and the notification appears. This could be confusing, so user does not know if the action was correctly performed

  • Cut option is only available for one item. Adding it to multiselection list

hosy added 3 commits July 20, 2021 16:06
…presentation

- prevents filenames
- handles txt pasteboard
- removed extra internal pasteboard
- moved pasteboard items to NSItemProvider
- allows copy items between accounts
- added download progress to copy action
@hosy
Copy link
Collaborator Author

hosy commented Jul 21, 2021

@jesmrec I refactored the pasteboard code and you can give it a new try.

After the 2nd round, pending stuff:

  • Checking if pasted items could preserve the original name

Items now preserve the original filename

  • Folders (not) pasted to different account in the same app.

At the moment folders cannot be pasted/cut between accounts. This needs more work but should be addressed in a new issue.

  • Question about the size of the copied content

Good question, this needs further research.

Some improvements that could be addressed in separate issues, as new iterations:

  • Pasting text from clipboard: selecting format to paste the text (report (5) )

Pasting text from clipboard now works as expected (only one file will be added)

  • Add a spinner/waiter while the items are copied: if many items are selected to be copied to clipboard, some time runs till everything is copied and the notification appears. This could be confusing, so user does not know if the action was correctly performed

I added a progress UI, when items will be copied.

  • Cut option is only available for one item. Adding it to multiselection list

Cut option is now available for multiselection list

@jesmrec
Copy link
Contributor

jesmrec commented Jul 21, 2021

Feature is approved.

But, it is approved under known limitations:

  1. It is not posible to...
  • ... copy folders between accounts
  • ... copy folders to external provider
  • ... execute Cut action between accounts
  • ... execute Cut action against an external provider
  1. Clipboard size to copy needs extra research as commented here. If the content copied inside the clipboard is great (pretty difficult to quantify, it depends on the device and its available storage), maybe the actions will not work fine, or the app gets stucked.

Test plan with all test cases designed and executed in the top message

I will create issues for the known limitations.

@hosy
Copy link
Collaborator Author

hosy commented Jul 22, 2021

@jesmrec answering your questions:

Feature is approved.

Thanks

But, it is approved under known limitations:

  1. It is not posible to...
  • ... copy folders between accounts

Yes, this is at the moment not supported by the SDK. We can create a new issue for that problem.
I can add an alert view, when a user copied a folder and want to paste it to another account?

  • ... copy folders to external provider

This is not supported by the iOS system at the moment.

  • ... execute Cut action between accounts

This is working for files (expect for folders).

  • ... execute Cut action against an external provider

This is not supported by the iOS system at the moment and also makes no sense in my opinion.

  1. Clipboard size to copy needs extra research as commented here. If the content copied inside the clipboard is great (pretty difficult to quantify, it depends on the device and its available storage), maybe the actions will not work fine, or the app gets stucked.

Yes, needs research.

Test plan with all test cases designed and executed in the top message

I will create issues for the known limitations.

@jesmrec jesmrec added the Approved by QA Approved by QA label Jul 22, 2021
@hosy hosy merged commit 319588f into milestone/11.7 Jul 22, 2021
@delete-merged-branch delete-merged-branch bot deleted the feature/pasteboard branch July 22, 2021 08:06
hosy added a commit that referenced this pull request Jul 29, 2021
* - set new version number/build number
- enabled beta build

* Configuration documentation updated

* fixing toolbar color value for branded clients (was overridden before)

* [feature/six-digits-passcode] 6 Digits Passcode (#1000)

* [feature/filename-webui-layout] Web UI Filename Layout (#1001)

* #968 Adopt the filename layout to the new WebUI:
- bold file name 17px
- nomal file extension 15px

* added changelog entry

* Calens changelog updated

* changed from bold to semibold

* fixed CR findings:
- renamed methods
- using UIFont.preferredFont instead of fixed font size points

* Calens changelog updated

Co-authored-by: hosy <[email protected]>

* fixed segmented control tint color theming

* [feature/pasteboard] Pasteboard Actions (#514)

* Naming improvements based on latest SDK:
- uploads use new OCCoreOptionAutomaticConflictResolutionNameStyle option to automatically resolve naming conflicts during upload
- Duplicate action uses the new OCCore name suggestion API to
	- determine naming conflicts and resolve them automatically
	- match the name style of the file to duplicate, f.ex.
		- duplicating "File copy.jpg" will create "File copy 2.jpg"
		- duplicating "File (1).jpg" will create "File (2).jpg"
		- duplicating "File Kopie 2.jpg" will create "File Kopie 3.jpg"
- folder creation uses the new OCCore name suggestion API to
	- detect naming conflicts beforehand
	- pre-fill the new folder name name with an unused name directly

* #393 added an activity indicator which will be shown, if offline copies will be deleted to give the user a UI feedback

* - Change SDK branch to master

* #76 user can import files using the iOS share sheet. all file types are accepted.

* - using new SDK with fixes for requesting core
- minor code fixes

* - Log device, version and locale information at the beginning of every log file using new SDK protocol
- Show SDK commit hash in Settings

* fixed crash, changed bookmark name to shortname

* changed UIAlertViewController for account selection to CardViewController style for testing

* fixed icon badge creation for fastlane lane In-House Enterprise IPA generation

* install librsvg for fastlane via sh

* moved import file code to own class

* added option to automatically resolve name conflicts, if file name already exists

* Changed app version to 1.1.0

* enabled beta build and warning

* Keep the gallery alive after doing some action over an item (#447)

* Rename updates the currently visible file in the gallery

* Keeping gallery alive fix

- Don’t pop to the root view controller on destructive action by default
- Only pop to the root if all items have been deleted
- Select next item upon deletion of the current ones
- Update UIPageViewController data source if number of items changes due e.g. to duplication or deletion

* Small fixes

* Fixed review findings

* Media player implemented using AVKit (#429)

* Media player implemented using AVKit

* Added possibility to stream audio / video

* Improved error handling

* Fixed review finding

* Added background audio, airplay and PiP mode

* Propagating safe area to the main view of AVPlayerViewController

* Another fix for safe area considering layout of AVPlayerViewController

* Updated to latest develop version of SDK

* Display error message in case file couldn’t be preview e.g. due to file corruption (#427)

* Fixed UI test for creating folder, need to return a fixed name, because suggestion is not working in tests

* added a description header and changed typo

* - fixed showing the directory picker controller, after the card view was dismissed
- use the current development SDK

* Version Bump to 126

* - create local copy of import file, if needed
- fixed create folder action
- delete local copy, if needed
- code review changes

* - improved duplicate item deletion behaviour
	- add additional safeguard so duplicate files are only deleted if they were actually duplicated (previously non-duplicate files could be removed if duplication or folder creation failed)
	- remove temporary container folder, too
- add 2 second delay before returning the core to give the core an opportunity to schedule the upload on a NSURLSession

* Version Bump to 127

* changed back signing identity

* Version Bump to 128

* added required CFBundleTypeName key

* Version Bump to 129

* updated changelog

* use formSheet presentation style for the iPad when showing the document picker

* Fix for images not being displayed in the gallery

* Version Bump to 130

* Version Bump to 131

* [fix/sharing-search] Fixed min length for searching sharing users (#455)

* #454 used correct comparator for sharingSearchMinLength and set a new default value, if capability does not exists

* added a constant for defaultSharingSearchMinLength value

* - Update ios-sdk to address finding (1) in ios-app #446

* Preventing updating UI in DisplayViewController while item is being changed

- Not calling present(item:) while meta data of OCItem is being updated
- Elliminated an assumption in MediaDisplayViewController that render renderSpecificView() can be called only once.
- Making sure that AVPlayer is not re-created upon changes in OCItem but that rather AVPlayerItem is replaced.

* Fixed a small warning

* Tried to improve gallery logic concerning items modification

- Watch out if the modification does still includes the item with local ID of the currently visible item.
- Take care of failed move but watching out for reappearing items

* Added a setting allowing to decide user if media files shall be streamed

* - Add debug output to Display*ViewController
- Fix SwiftLint warnings

* Fixed handling of deleted / moved item in the gallery

* Another small fix for handling of failed item move

* Version Bump to 132

* Added LSSupportsOpeningDocumentsInPlace key to Info.plist

* Fixed Info.plist and added LSHandlerRank property

* Fix for #455 issue: no search request triggered

* show always the account selection sheet and added a note, that only one file can be imported at once

* use securtiy scoped file operation for importing a file

* Fix for the PR #447 (keep gallery alive) (#465)

* Keeping track of individual OCItems in DisplayViewController instances

But loading the list of items in the gallery only once and not reacting to any changes (moving, deleting)

* Removed query stop call in DisplayHostViewController

Since this query is not created there but just passed from the parent view.

* Removing more button if the currently viewed file got moved or deleted

* Fixed updating UI after renaming current item

* Fixed a warning

* Fixed issues in the gallery

* - Minor fixes

* [feature/bundle-import] Add support for certain bundle formats to the share sheet (#471)

* - Add support for import of bundle-based document formats through transparent zipping
	- moved previous FileProvider functionality into ownCloudApp.framework
	- new method OCCore.importItemNamed() ensures correct handling of bundle-based document formats
	- fixes finding (4) in #445
- Changed LSSupportsOpeningDocumentsInPlace to NO address two issues:
	- the app is now listed as "Copy to ownCloud" (previously "Open in ownCloud", which was technically not correct)
	- previously, documents using "Open to ownCloud" would no longer open in f.ex. Pages, but in the ownCloud app when selected in the Files app

* - Refactored ImportFilesController around using NSFileCoordinator, fixing finding (1) in #471

* changed path for moved Info.plist for target ownCloudAppFramework

* Version Bump to 133

* [feature/sort direction] Allow sorting asc/desc for all sort methods (#474)

* #470 added change sort direction asc/desc for all sort methods

* - removed comments
- changed translation keys

* fixes sort comparator for sort method "shared"

* use correct images for sort direction asc and desc

* updated choosing sort method from a tableView/Popover instead of a UIAlertViewController

* set maximum width for popover and set arrow direction

* Added show sort direction in UISegmentControl

* - make sure arrows of popovers appear in same color as the popover contents
- fix issue where direction was toggled during setup of SortBar (setting of initial value from user defaults), so logging in, logging out and logging in would change the sort direction
- reset sort direction to ascendant when switching sort methods
- SortMethodTableViewController
	- remove extraneous instance lets
	- enforce row height used for table view height calculation
	- change table view height calculation to account for the last divider (which could look a bit off if present)

* tap on segment control was fired twice, because there was a missing check, if selected segment changed

* fixed changing width on selecting an other segment

* fixed jumping width of sortSegmentControl items by setting the same width for all items by calculating the longest width before

* [fix/index-bar] Make Index Bar available in general file list (#469)

* #413 show index bar in general file list
- Index bar is only visible, if sorting is "Sort by name"
- Index bar is only visible, if more than one different letters are available
- Index bar is only visible, if more items are available as visible

* - adopt to new sort methods
- reverse index list, if sort direction is descendant

* Version Bump to 134

* Version Bump to 135

* Version Bump to 136

* Change behaviour of default expiration date (#476)

* Enabling expiration date switch if default expiration date is configured

Checking files_sharing.public.expire_date.expire_date.days server capability. If it is set then it is used as default date and the switch is enabled even if the expiration date is not enforced.

* Fixed QA issue (1)

* Fixed QA finding (2): limiting the date range if expiration date is enforced

* Implement UIKeyCommands to access all actions and navigate through table views via hardware key shortcuts

* Permission increasing UX in reshares (#467)

* Not showing permissions which can’t be elevated

Affecting items shared with the current users which are being reshared

* Fixed an issue with wrong permissions displayed when creating public link

- Checking if permissions can be elevated for the public link
- Only viable options are displayed

* Fixed a trailing whitespace warning

* Fixed another compiler warning

* Inhereting permissions from the share when creating public link

* Fixed QA finding (4)

* Fixed QA finding (1)

* Fixed QA finding (2)

* fixed table selection in StaticTableView with key command

* Version Bump to 137

* - moved KeyCommands extension into own file
- added new commands

* [feature/itempolicy] Item Policy / Available Offline support (#456)

* Available Offline Support
- new MakeAvailableOfflineAction and MakeUnavailableOfflineAction actions
- new Available Offline Library section
	- easy access to all files available offline via Available Offline
	- new ItemPolicyTableViewController and ItemPolicyCell prepared for reuse for presenting other item policies
- Design: added SVG/TVG/PNG icons and cloud symbols for make available offline / make unavailable offline

ClientItemCell changes
- new ClientItemResolvingCell can resolve item from LocalID or path, new superclass of ShareClientItemCell and ItemPolicyCell
- string to use as content for title and detail are now provided via titleLabelString(for:) and detailLabelString(for:), updated via updateLabels(with:), making customization easier
- support for new available offline badge

ProgressSummarizer changes
- changed ProgressSummary from a struct to a class, addressing issue #451 ("Waiting for server response..." message sometimes not disappearing)

* - Adopt OCClaim to ensure files aren't deleted while they are being used in
	- DisplayViewController
	- DownloadItemsHUDViewController
	- FileListTableViewController
	- File Provider
- Add new Storage Settings to allow control over how long unused local copies are kept around
- DisplayViewController now uses OCClaims to ensure files aren't deleted while they are being viewed
- ClientItemCell uses new SDK APIs to
	- show "available offline" badges for folders
	- to indicate if an item has been made available offline directly (solid icon) or indirectly (dimmed icon)
	- keep the cloud status icon updated following changes
- Change "+" icon to "•••" icon in file list, rename plusButton to folderAction everywhere
- Action.provideAlertAction() now applies padding to images if necessary so their text aligns at the same horizontal offset
- Fix typo in BookmarkInfoViewController and change wording from "Delete Offline Copies" to "Delete Local Copies" for consistency
- Update swiftlint.yml to silence ownCloudScreenshotsTests warnings

* - Update ios-sdk

* - Make changes requested in code review, adding missing localizable strings and removing extraneous code

* - Update SDK

* - Add downloadTriggerIdentifier and availableOfflinePolicyCoverage as properties that trigger the update of an item's visual representation (addressing finding 6 in #456)
- Update make available offline action to use new convertExistingLocalDownloads option (addressing finding 2 in #456)
- Update SDK (addressing finding 5 in #456)

* Items in moved Available Offline folders should no longer be cleared locally and re-downloaded, following under the hood improvements in the ios-sdk

* - Add initial support for cookies via SDK update

* - Temporarily disable some UI tests broken due to lack of database running

* - Fix issue (11) in #456 by updating the lastUsed date of already downloaded items when they are viewed

* - Added slider type to StaticTableViewRow
- changed storage settings to allow picking a timeframe via slider rather than a list of time frames

* - Change text from "Remove" to "Make unavailable offline" in Quick Access > Available Offline list (addressing issue (9))

* - Update BookmarkInfoViewController to provide users with the choice to also remove local copies of items marked as Available Offline, issuing a warning and giving the users a way to cancel, addressing issue (12) in #456.

* - Update ios-sdk
- Take advantage of the new key-value store and OCCoreSkipAvailableOfflineKey to make sure available offline files are not immediately downloaded again after clearing the vault from local copies of available offline files.
- Logging into an account removes any previously set OCCoreSkipAvailableOfflineKey from the key value store
- Move Bookmark locking to an OCBookmarkManager category (next stop: use the new key-value store and bake support for locking bookmarks directly into the SDK)

* - Add missing localization

* Update SDK with latest fixes

* - Fix SettingsTests bugs

* - Apply additional fixes to UI tests and disable the remaining failing ones

* - Wait for connection initialization to complete before requesting sharing info
- Use latest SDK with cookie filtering

* - Turn off Request/Response log tags only filter in project file

* - Disable cookie support

* - Address (13) via SDK update

* link against newest development SDK

* Fixes for UI testing:
- use new SDK version that returns errors if the SQLite DB has not been opened
- MockOCCore.state now returns .running to avoid queueing of PROPFINDs

* - Fix unit tests

* - UI testing: make MockOCCore.state no longer return .running due to side effects
- fixed conflicting constraints in two places

* - force-delete bookmarks in UI tests if timeout has passed, log error but don't assert in that case
- handle case where a swipe on table cells in DeleteBookmarkTests triggers deletion outright

* - Further hardening of SettingsTests against timing differences local vs. CI

* - make SettingsTest.testCheckMoreItems() more robust

* Version Bump to 138

* [feature/reorder buttons] Reorder Navigation Bar Actions (#478)

* Available Offline Support
- new MakeAvailableOfflineAction and MakeUnavailableOfflineAction actions
- new Available Offline Library section
	- easy access to all files available offline via Available Offline
	- new ItemPolicyTableViewController and ItemPolicyCell prepared for reuse for presenting other item policies
- Design: added SVG/TVG/PNG icons and cloud symbols for make available offline / make unavailable offline

ClientItemCell changes
- new ClientItemResolvingCell can resolve item from LocalID or path, new superclass of ShareClientItemCell and ItemPolicyCell
- string to use as content for title and detail are now provided via titleLabelString(for:) and detailLabelString(for:), updated via updateLabels(with:), making customization easier
- support for new available offline badge

ProgressSummarizer changes
- changed ProgressSummary from a struct to a class, addressing issue #451 ("Waiting for server response..." message sometimes not disappearing)

* - Adopt OCClaim to ensure files aren't deleted while they are being used in
	- DisplayViewController
	- DownloadItemsHUDViewController
	- FileListTableViewController
	- File Provider
- Add new Storage Settings to allow control over how long unused local copies are kept around
- DisplayViewController now uses OCClaims to ensure files aren't deleted while they are being viewed
- ClientItemCell uses new SDK APIs to
	- show "available offline" badges for folders
	- to indicate if an item has been made available offline directly (solid icon) or indirectly (dimmed icon)
	- keep the cloud status icon updated following changes
- Change "+" icon to "•••" icon in file list, rename plusButton to folderAction everywhere
- Action.provideAlertAction() now applies padding to images if necessary so their text aligns at the same horizontal offset
- Fix typo in BookmarkInfoViewController and change wording from "Delete Offline Copies" to "Delete Local Copies" for consistency
- Update swiftlint.yml to silence ownCloudScreenshotsTests warnings

* - Update ios-sdk

* - Make changes requested in code review, adding missing localizable strings and removing extraneous code

* #477 reorganize navigation bar with create and more button and moved select button to sort menu

* added safeAreaLayoutGuide.rightAnchor

* - Update SDK

* changed "Select" image

* - Add downloadTriggerIdentifier and availableOfflinePolicyCoverage as properties that trigger the update of an item's visual representation (addressing finding 6 in #456)
- Update make available offline action to use new convertExistingLocalDownloads option (addressing finding 2 in #456)
- Update SDK (addressing finding 5 in #456)

* Items in moved Available Offline folders should no longer be cleared locally and re-downloaded, following under the hood improvements in the ios-sdk

* - Temporarily disable some UI tests broken due to lack of database running

* - Fix issue (11) in #456 by updating the lastUsed date of already downloaded items when they are viewed

* - Added slider type to StaticTableViewRow
- changed storage settings to allow picking a timeframe via slider rather than a list of time frames

* - Change text from "Remove" to "Make unavailable offline" in Quick Access > Available Offline list (addressing issue (9))

* - Update BookmarkInfoViewController to provide users with the choice to also remove local copies of items marked as Available Offline, issuing a warning and giving the users a way to cancel, addressing issue (12) in #456.

* addresses:
- missing multiselection UI in toolbar
- missing select all / deselect all button in navigation bar

* set if selectButton should be shown

* disable temporary some tests due to bitrise failures

* using correct SDK after merge

* fixed changed selectButton variable name after merge

* - added missing commands
- added photo album, photo selection
- fixed deselecting row

* fixed folder actions commands

* added sharing and links key commands

* - added key commands for LibrarySharing
- added localization

* - added missing key commands
- fixed keep selection, if FileList was reloaded
- fixed localization
- implemented copy and paste commands
- implemented new selection commands (letters, top, bottom, page)

* added commands for bookmark selection

* Determine, if the internal pasteboard is the current item and use it

* - implemented "Cut"-Command, "Favorite"-Command
- implemented Next, Previous Command for image gallery and PDF viewer
- fixed Sort-Type Command Key, because of system conflicts
- changed command key for adding a new account

* - refactored code to own actions
- cleanup code
- use extensions for key commands

* - fixed code review findings
- moving pasteboard actions copy, import and cut to separate PR

* Added new actions for Pasteboard: Copy, Cut, Import

* - updated version and build number
- enabled beta warning

* - add support for direct theming of progress colors also for light and dark mode

* using URL keyboard for configure up a branded server URL

* localized missing string in sharing receipient search field

* [fix/auto-page-scrolling] Auto Enable Page Scrolling in Detail View (#920)

* [feature/align-bookmark-names] Align bookmark names with Android and web (#922)

* [feature/prevent-sleep-detail-view] Presentation Mode (#893)

* [feature/french-localization] French Localization (#927)

* #4450 added french localization with latest translated strings from transifex

* Calens changelog updated

* Calens changelog updated

Co-authored-by: hosy <[email protected]>

* [feature/fileprovider-passcode-UI] File Provider Passcode UI  (#881)

* [feature/drag-drop-setting] Setting to disable dragging files inside the app (#643)

* #581 Added new setting to disable dragging files inside the app

* addressed CR findings and changed the title and the default value

* changed variable name for prevent dragging files and folders

* - update SDK
	- adapt BookmarkViewController to latest OCBookmarkManager usage patterns
- improve Illustrator ".ai" support
	- override for failing iOS MIMEType -> UTI conversion in FileProvider (-> fix grayed out items in Files app)
	- add "application/illustrator" mime type to PDFViewerViewController to allow viewing Illustrator files in-app
- improved sorting
	- replace "type" with "kind" to match Files app and change method:
		- sort by type (folder / file)
		- if identical: sort by suffix (with fallback to MIMEType if no suffix is available)
		- if identical: use "_various" for comparison

* Using creationDate instead of modificationDate (#668)

Change shall avoid upload of duplicated photos since e.g. modificationDate can change if the photo was edited.

Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>

* Update documentation to reflect changes in instant photo upload (#677)

* Document task scheduling

This relates to https://github.com/owncloud/ios-app/pull/484 and
https://github.com/owncloud/docs/issues/1900.Document task scheduling

This relates to https://github.com/owncloud/ios-app/pull/484 and
https://github.com/owncloud/docs/issues/1900.

* Document task scheduling

This relates to https://github.com/owncloud/ios-app/pull/484 and
https://github.com/owncloud/docs/issues/1900.Document task scheduling

This relates to https://github.com/owncloud/ios-app/pull/484 and
https://github.com/owncloud/docs/issues/1900.

* Update docs/modules/ROOT/pages/ios_task_scheduling.adoc

Co-Authored-By: Phil Davis <[email protected]>

* Update docs/modules/ROOT/pages/ios_task_scheduling.adoc

Co-Authored-By: Phil Davis <[email protected]>

* Changed documentation to reflect changes in instant photo upload

Co-authored-by: Matthew Setter <[email protected]>
Co-authored-by: Matthew Setter <[email protected]>
Co-authored-by: Matthew Setter <[email protected]>
Co-authored-by: Phil Davis <[email protected]>
Co-authored-by: Michael Neuwert <[email protected]>

* [feature/improve-public-link-creation] Improve creation of public link (#673)

* #671 Improve creation of public link
- enable dragging public link
- added clear button on public link name
- copy link to clipboard, after link was created

* - added in-app notifications to the app
- show a notification if a public or private link was copied to the clipboard

* implemented own notification banner class to show in-app notifications without a need of local notification permissions

* resolved crash

* - implemented own animator class
- renamed class
- added completion handler on dismissal
- added shadow

* [fix/audio-stop-playing] Stop playing Audio when item loosing focus (#686)

* #683 fix stop playing audio, when scrollview was swiped to next item, play again, when come back to the audio item

* moved code to given delegate methods

* [fix/round-shape-button] Fix round shape for pin code buttons (#689)

* #654 bring back the round shape to the button, after changes in super class

* added a new enum type to set corner radius with enum values

* [feature/folders-first] Option to show folders at the top (#680)

* - add "Show folders at the top" option in Settings
- fix DisplaySettings change propagation within the same process (relevant when using multiple scenes on iPadOS)
- restructure SortMethod and add support to show folders at the top for all sort methods

* - Update CONFIGURATION.md to include new and previously undocumented configuration options

* [feature/resign-script] Resign Script  (#690)

* Resign script with instructions how to resign an IPA file with an other certificate and provisioning profiles

* Using correct markdown format

* changed formatting for better readability

* Updated readme file

* changed new lines

* changed formatting

* changed formatting

* changed wording

* - changed folder structure
- added a parameter check for the signing method

* - show available certificate identities in usage
- check provisioning profiles fingerprint if matches with signing identity fingerprint

* [iOS] Open private link in app (#609)

* Adapted to latest SDK changes in develop branch

* Implemented support for private links

- Support for opening links using custom URL scheme owncloud://
- Support for associated domains added
- Test associated domain configured

* Some small fix for presenting linked item

* Fixed an issue with universal links not being opened if the app was terminated in iOS13

* Triggering item download if necessary when core connection status changes to online

* Added a comment requested in code review

* Fixed build issues

* Iterating through multiple bookmarks possibly related to same URL

* Fixed review finding

* Fixed various swift Lint findings

* Fixed minor swift lint warnings

* - Licensing: add finishTransaction call for SKPaymentTransactionStateRestored transactions

* Fixes an issue with private link not being opened

This would happen in case user has kept opened a card with actions (via 3 dots button)

* Removed dead code and ensured that no item is opened at next launch if user returns to server list

* Handling going back in state restoration

* Added error message in case link can’t be resolved

* Swiflit trailing space warnings fixed

* - Update SDK to resolve issues

* Made download dependant on state of the connection

More reliable download mechanism for file previews:
- Trigger download only if network connectivity is available
- React to connection state changes

* Fixed an issue with owncloud:// schemed links not opened

* Trying to trigger download when network becomes available

* Fixed swift lint warning

* Avoiding multiple update of display view state

* Updated to the latest SDK

* Fixed an issue with DispatchGroup.leave() sometimes not being called

* Adapted for a new connection status .connecting

* Some swift lint warnings fixed

* Some refactoring in the DisplayViewController

* Tried to make DisplayViewController more maintenance friendly

- Code restructuring
- Handling of connection status
- Re-worked view state handling
- Tried to unwind some deeply nested hard to understand if conditions

* Included a fix in the SDK which brings back missing parent OCItems

* Showing “Connecting…” before OCCore switches connection status to online

* Several small changes

* Improved private link handling

- Refactored to avoid unnecessary code duplication
- Improved error messages
- Showing progress HUD while link is being resolved since it might take time espeially with multiple configured bookmarks and so on

* Updated to the develop/latest SDK revision

* Changed how error message is chosen if the item for private link is not found

* Using OCCoreConnectionStatusSignal instead of OCCore.connectionStatus

* Fixed “Connecting…” visual glitch and added spinner shown while connecting

* Fixed opening of app registered scheme on iOS13

As well as eventually a bug concerning file imports in case the app was not launched

* Added a delay for iOS 12.x for the private link resolution

So when the root view controller is instantiated it might already start some operations like opening a root of bookmark etc. Otherwise it would probably require some complex coordination managed centrally. On iOS 13 such hack would be unnecessary since there UIApplicationDelegate callbacks are not used anymore.

* Improvement for QA finding

If the bookmark to which private link has pointed is not configured,  wrong error message has been shown

* Fixed an issue with incorrect error message being shown

In case item was removed

* Fixed an issue with file previews

- Not downloading by default if file can’t be previewed
- Downloading and opening files which can be previewed by QLPreviewController

* Fixed an issue with renderSpecificView() called at inapropriate time

Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Felix Schwarz <[email protected]>

* new version and build number

* - Update SDK to add latest SDK HTTP request/response logging improvements

* Fixed some issues with VoiceOver / accessibility labels (#701)

* Set accessibility properties on the actions menu drag handle

* Fixed accessibility label for the favorite / unfavorite button

* Fixed wrong order of accesibility labels for star button (favorite)

* Added accesibility label for button activating multiple item selection

* Added proper accessibility labels to the toolbar items

* added mouse pointer hover effect to drag view

Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>

* - Update SDK to add preliminary/experimental TUS support

* show view controllers as formSheet, because fullscreen is to large for this small view on the iPad

* Take Photo or Video and Upload (#707)

* First draft of take photo and upload feature

* Don’t allow image editing when taking new pic

* Next iteration of “Take photo or video”

- Preserving image meta-data
- Conversion HEIC -> JPEG depending on media upload settings
- Conversion MOV -> MP4
- Localized action name

* Fixed media naming and added some error logs

* Some cosmetic code changes

* Changed keyboard shortcut to Cmd+Shift+3

* Added logs to camera upload action

* Added progress hud in case photo / video export would take longer

* Added more logs

* Added a check for HEIC support

Warning added to the log in case HEIC is not supported

Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>

* - update SDK to include JSON logging options
- add Xcode scheme env var templates to simplify testing

* [feature/contextual-menu] Contextual Menu for File List Items (#718)

* Correct some typos (#620)

Fix small errors in typos

Co-authored-by: Matthias Hühne <[email protected]>

* add extra check in release checklist, to avoid missing translations in some languages (#640)

Co-authored-by: Matthias Hühne <[email protected]>

* Fix docs build errors (#666)

Co-authored-by: Matthias Hühne <[email protected]>

* #717 Implemented a contextual menu for the file list which includes the same items as the more menu card

* - UIMenu title removed
- use table row label color as image tint color (UIMenu background color is wrong in simulator and dark mode)

* solved tint color problems for UIAction icon

* fixed wording and localization

Co-authored-by: Carlos Damken <[email protected]>
Co-authored-by: Jesús Recio <[email protected]>
Co-authored-by: Matthew Setter <[email protected]>

* Fix for the resign script (#721)

- Do not embed entitlements into library binaries when signing frameworks. Otherwise leading to validation issue with AppStore Connect.
- Set NSExtensionFileProviderDocumentGroup to appropriate app group. This was missing

Co-authored-by: Michael Neuwert <[email protected]>

* Another fix for resigning script

Delete tmp.plist after it is no longer being used, otherwise leads to ITMS-90048 Bundle is invalid.

* Support changing the format of uploaded image file names (#687)

* Improved various aspects of media uploads

- Changed a way how assets are exported which photo uploads significantly faster
- Refactored PHAsset+Upload.swift extensions which is now way more readable.
- Improved naming of assets
- Changed default naming of uploaded assets to the scheme legacy app was using
- Added an option allowing to preserve “original” asset names as used by iOS photo library
- Prepared code for future enhancments (exporting additional assets like RAW images and live photo video clips

* Localization of strings and other tiny changes

* Made a mechanism deriving original media asset name more reliable

* Fixed requesting image data

* Using PHAsset.creationDate as lastModifiedDate when uploading media asset

The timestamp is then set into corresponding property in the WebDAV request allowing to have proper file creation date when uploaded to oC server instance.

* Picking edited version of the photo when uploading by default

* Moved files around to have a cleaner structure in Xcode

* Using PHAssetResourceManager for export whenever possible

* Fixed small issues

* Corrected some comments

* Fixes issue (1) found in QA testing

Fixed duplicated .MP4 extension when uploading videos

Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Felix Schwarz <[email protected]>

* Fixed 1.4 branch

* - update SDK to include tus creation-with-upload support
- add additional env parameters for logging, auth methods

* - update SDK to fix infinite loops when hitting a redirect or certificate issue, eliminating redundant issues, respecting and remembering choices to either retry or put the connection offline

* [fix/various] Various fixes (#713)

* - FileProvider: extend hardcoded MIME/Suffix -> UTI maps so files of these types are properly displayed in the Files app

* - add new ItemContainer protocol allowing the retrieval of an item from another object
- UIImageView+Thumbnails
	- fix item version comparison for thumbnails requested from the SDK (using ItemContainer)
	- add item version comparison for thumbnails generated locally via QLThumbnailGenerator (using ItemContainer)

* - Update SDK

* - sharing items will be shown as subsection in contextual menu
- only show sharing items, if sharing is enabled and core is online

* changed image for rename menu item to pencil on > iOS 13

* Fixes for markup and PDF viewer (#730)

* Activate editing mode in viewDidAppear()

Sort of a hack but there is apparently no other way of achieving this

* Display more button in the PDF viewer. Preserving already configured UIBarButtonItem instances when adding more button.

* Re-rendering content if it has got modified locally

E.g. user could open markup editor and scribble on picture or PDF. Without this fix, he would need to go back to the file list and re-open document to see the changes.

* Fix for auto-tapping markup button

Co-authored-by: Michael Neuwert <[email protected]>

* fixes malformed file

* fixed code signing:
- use entitlements from app binary instead of provisioning profile
- set app values from provisioning profile to entitlements file
- only set NSExtensionFileProviderDocumentGroup for File Provider extension

* Include new dialog to encourage users to add a new app review in App Store  (#728)

* Added all bits and pieces for app review request

* Fix for a review task not being run

* Changed the way review prompt is presented

* Fixed review findings

* Corrected sign of time interval properties

* Another sign for time interval corrected

* Corrected the way review is requested

Co-authored-by: Michael Neuwert <[email protected]>

* [feature/auth-migration] Authentication Method Migration (#682)

* - OCBookmark+Extension
	- clean up OCBookmarkUserInfoKeys and move them to their own extension
	- add new OCBookmark.scanForAuthenticationMethodsRequired to mark bookmarks as needing a scan for available auth methods before editing
- BookmarkViewController
	- add support for OCBookmark.scanForAuthenticationMethodsRequired
- ClientRootViewController
	- add scan for available authentication methods once when encountering an authorization error
	- rename "Ignore" to "Continue offline"
	- extend ClientRootViewControllerAuthenticationDelegate to include preferredAuthenticationMethods
	- prepare for inline authentication data (username + password - or -  tokens) gathering in the next step

* - Client: add support for handling auth failures inline, without dropping out of the connection:
	- ClientAuthenticationUpdater
		- offers high-level abstraction
		- ties together token-/web-based auth UI and password-based auth UI (ClientAuthenticationUpdaterViewController)
	- ClientAuthenticationUpdaterViewController
		- view controller specialized in requesting a password from the user

* - Update SDK

* - ClientRootViewController: add support for OCCoreConnectionStatus.connecting

* - Update SDK

* - ClientAuthenticationUpdater: add check for .preferredAuthenticationMethodIdentifiers to .canUpdateInline
- ClientRootViewController: add support for plain HTTP bookmarks to authentication method scan
- add additional logging

Co-authored-by: Matthias Hühne <[email protected]>

* iOS13 gesture based multiple selection (#734)

* iOS13 gesture based multiple selection

- In file list view (2 finger pan)
- In photo selection grid view (1 finger or 2 finger pan)

* Disable gesture based multi-selection if prevent dragging optin is active

* changed option title and added a subtitle with a short description

* using better wording

Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>

* [feature/emm] EMM version support (#706)

* Add EMM support:
- new OCLicenseEMMProvider
	- unlocks all pro features if the main bundle ID (as found in the app's or app extension's Info.plist) ends with either "-emm" or ".emm"
	- OCLicenseEMMProvider.isEMMVersion provides information on whether the app is an EMM version
- Licensing Setup: if the app is an EMM version, no App Store IAP provider is set up/added
- Settings: for EMM versions, don't show the "Pro Features" or "Purchases" entries

* - added new fastlane lane to build a emm version of the iOS app
- using the new fastlane file from migration bundle-id branch

* init fastlane deliver action to get the current metadata from the app store to prepare for different apps uploads (regular iOS app and EMM version)

* new fastlane action to parse the release notes plist file

* - new fastlane lane too upload regular and emm iOS build to the AppStore, before release note, screenshots can be created automatically
- new fastlane action to generate the release notes from the ReleaseNotes.plist
- added new options for fastlane lane

* show suffix "EMM" on settings, support mail, when the app was a EMM build

* - fixed signing issues
- changed copyright date
- removed app name metadata template, because of conflicts when using different apps
- updated release notes

* - added app icon for emm version
- exchange app icon with emm version before building
- added a description.txt metadata file for emm app version

* - added app icon for emm app (AppStore Connect upload)
- moving description and app icon for metadata upload
- uncomment code after testing

* new app icon for emm version

* fixed app icon dimensions

* using function from OCLicenseEMMProvider

* enable binary upload on deliver action for emm build

* using newest SDK, to prevent using UIWebView

* updated app version number

Co-authored-by: Matthias Hühne <[email protected]>

* Removed test configuration of associated domains

* Instant Uploads part II + more (#714)

* Improved various aspects of media uploads

- Changed a way how assets are exported which photo uploads significantly faster
- Refactored PHAsset+Upload.swift extensions which is now way more readable.
- Improved naming of assets
- Changed default naming of uploaded assets to the scheme legacy app was using
- Added an option allowing to preserve “original” asset names as used by iOS photo library
- Prepared code for future enhancments (exporting additional assets like RAW images and live photo video clips

* Localization of strings and other tiny changes

* Made a mechanism deriving original media asset name more reliable

* Fixed requesting image data

* Using PHAsset.creationDate as lastModifiedDate when uploading media asset

The timestamp is then set into corresponding property in the WebDAV request allowing to have proper file creation date when uploaded to oC server instance.

* Picking edited version of the photo when uploading by default

* Moved files around to have a cleaner structure in Xcode

* Using PHAssetResourceManager for export whenever possible

* Fixed small issues

* Corrected some comments

* Some small refactoring changes

* Add support for cellular transfer settings
- rename StorageSettingsSection to DataSettingsSection to extend scope
- add row and cellular transfer summary to DataSettingsSection
- add new CellularSettingsViewController to present and allow control over cellular settings

* - Update SDK with cellular switch improvements

* - Update SDK to adjust concurrency budgets for sync action so that wifi-only up-/downloads can't block cellular up-/downloads entirely

* Moved media upload settings into separate view

- New view for media upload settings
- Split settings into sections: Export, Auto Upload
- Changed appearance of the Upload Path setting
- Code refactoring changes

* Fixed swift lint warning

* Added cellular switch for media uploads

* Separate configuration for video auto-upload

Added posibility to select different account / path for video auto-uploads

* Fixed small issue in video upload

* Improved the UX for photo album view in upload

- Improved the way thumbnails are requested
- While view is active thumbnails for albums are cached
- Added activity indicator which is shown while albums are being fetched from photo library

* Uploading media from iCloud

- Showing cloud based photo albums in the selection UI
- Ability do upload media which is not available locally

* Improved vide exporting code

* Showing camera roll first in list of albums

* Refactored the method for asset fetching into separate extension

* Media upload queue refactoring

- More modular design and more maintainable code
- Started using OperationQueue for individual imports and created dedicated Operation subclass

* Forgot to add license header

* Small tweaks in media upload queue

- Changed import queue QoS to utility
- Added another cancellation point for MediaUploadOperation

* Configured main thread checker not to stop on first hit

* Background media uploads

- Improved and extended settings UI
- Using both BackgroundTasks and background location to enable background media uploads
- Local notifications informing about background uploads

* Fixed few issues concerning auto-upload settings

* Fixed some issues and added cellular switch for videos

- Some code refactoring in PHAsset export code
- Calling the upload completion handler when placeholder item is created allowing MediaUploadQueue to proceed

* Fixed an issue with photoLibraryChangeDetected flag reset at inappropriate time

* Improved wording in background upload settings

* Refactored local notifications in separate category

* Fixed swift lint warnings

* Cleaned up code

* Improved task scheduling code

* Re-added AVAsset extension

* Don’t show auto-upload settings if no accounts are configured

* Small improvements

- Wording for background location on iOS13 changed
- Added cell identifiers for easier creation of UI tests

* Fix of UI test was necessary since settings have moved

* Additional test for auto-upload settings added

* Fixed remaining failing UI tests

* Implemented CLLocationManagerDelegate error callback

* Re-applied a fix lost during merging

* Changed a way location tracking is initiated

- Start when app is going into background
- Stop when app is foreground
- Switch from significant location to visit monitoring

* Fixed review findings

* Removed a WiFi requirement from pending media upload task

* Removed some unused code

* Added more logs

* - add experimental option to force-enable background NSURLSession usage in the app
- in code, change all instances of the usage of the term "master" to "main"

* Removed an option allowing background NSURLSession

* Change the way background upload settings behave

* Removed background upload settings

Commented out and added a TODO

* Fixed some swift lint warnings

* Fixed review finding

* Only show bookmark / path if instant upload is enabled

Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Felix Schwarz <[email protected]>

* [feature/cellular-switch] Cellular transfer options (#709)

* Correct some typos (#620)

Fix small errors in typos

Co-authored-by: Matthias Hühne <[email protected]>

* Add support for cellular transfer settings
- rename StorageSettingsSection to DataSettingsSection to extend scope
- add row and cellular transfer summary to DataSettingsSection
- add new CellularSettingsViewController to present and allow control over cellular settings

* - Update SDK with cellular switch improvements

* - Update SDK to adjust concurrency budgets for sync action so that wifi-only up-/downloads can't block cellular up-/downloads entirely

* - Update SDK with new prefixed-HTTP/single-line logging options
- Add new scheme options to selectively turn off single-line and prefixed HTTP logging

* - Update SDK to include logging improvements

* add extra check in release checklist, to avoid missing translations in some languages (#640)

Co-authored-by: Matthias Hühne <[email protected]>

* - Update SDK

* - in code, change all instances of the usage of the term "master" to "main"
- update SDK

* - removal of iOS 11 support (including SFAuthenticationSession) and updating to iOS 12+ APIs, adaption of docs where needed
- migration from MobileCoreServices to CoreServices
- drop deprecated serialization APIs in AppLockManager and FileProvider
- Cellular Options
	- display a message if cellular transfers have been disabled via MDM
	- adopt updated SDK to include cellular option improvements

* - Update message in Cellular Settings and dynamically show/hide it depending on whether the main switch is enabled or not

Co-authored-by: Carlos Damken <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>
Co-authored-by: Jesús Recio <[email protected]>

* [feature/app-version-clipboard] App Version Infos to Clipboard (#741)

* #740 moved app version infos from footer to a row to perform a copy action to the clipboard with app version infos

* - fixed localization
- removed unneeded completion block

* removed target ownCloud File Provider UI, because this was never used
Deletion of this target solves a problem when building with fastlane (code signing error)

* [fix/missing-file-list-actions] Implements missing actions in Quick Access file list (#743)

* #733 some actions like contextual menu, multiple selection and swipe actions was missing in the generic file list class. Moved missing code to super class or super-super class.

* moved table view row swipe actions to parent class

* [feature/message-queue] Message Queue to allow flexible handling of issues and messages (#662)

* - ClientRootViewController: do not add dot to the end of the short connection status description if it already has one
- UIImageView+Thumbnails: fix retain loop, leading to leaking an OCCore - and all possible issues relating to that

* - Update to latest SDK

* - Update SDK and adapt ProgressSummarizer for new event type

* - Fix core-stop-blocking share reload query retain loop via SDK update

* - add permission checks to CreateFolderAction, DeleteAction, DocumentEditingAction, MoveAction, RenameAction, ScanAction and UploadBaseAction, so they are unavailable if permissions aren't sufficient for their usage
- add alert to inform the user that no actions are available in cases where no actions are available for a folder and the user presses the folder action "+" button

* - Update SDK

* fixed QA finding (1) #623

* Fix for QA issue (2)

Media upload settings shall not allow selection of the upload folder which lacks required Permissions

* #623 QA finding (6) allow Markup it at least one permission is available and restrict save actions to available permissions

* - address issue (1) using ActionContext.query.rootItem

* OCCore+Extension.swift:
- remove items from OCShareQueries where they are not used by the core to avoid warnings in the log
- add option for partial matches to sharesSharedWithMe() via allowPartialMatch option

PublicLinkTableViewController.swift:
- take advantage of allowPartialMatch option
- determine share nested the deepest to determine permissions (fixes finding 84) in #632

* - Update SDK

* - avoid using ":", "/" and "\" in localized time stamp of auto-created file names for document scanning, using "-" instead
- update SDK

* - DisplayViewController
	- make sure content is updated as new versions become available (fixing #630)
- FileProvider
	- make use of new SDK OCClaimLockTypeRead to encourage automatic file updates for opened files
- SDK update

* - Update SDK

* - Update SDK

* - DisplayViewController:
	- update for local changes, tracking the last modified date of the local file
	- source:didSet: remove duplicate/out-of-sync checks

* - EditDocumentViewController: fixing tabs, refresh when source file changes
- DisplayViewController: clarify code, add debug output
- OCItem+Extension: new OCItem.contentDifferent(than:core:) to determine whether the content of an item has likely changed based on two OCItem, based on identifiers and size, adapt OCItem.displaysDifferent(than:in:) to use the new method.

* - Change method to refresh QLPreviewController to .reloadData() as it works more reliable

* - Update SDK

* - Initial support for message queue

* - wrote down UI integration ideas

* - ownCloudApp framework
	- NotificationManager: provides necessary infrastructure for local notifications
	- NotificationMessagePresenter: an OCMessagePresenter using local notifications
- File Provider
	- temporarily drop all early-error-catching code to allow debugging messages
- Client
	- ClientActivityViewController: added support for display of OCMessages
	- ClientRootViewController:
		- added notificationPresenter and issueMessagePresenter
		- added presentAlertAsCard(viewController:withHandle:dismissable:) to present view controllers using the alertQueue
	- rewrite MessageCell to use the new AlertView instead
- Messages
	- AlertView and AlertViewController provide alert views suitable for showing as a card or inline in a table
	- CardIssueMessagePresenter: an OCMessagePresenter presenting messages as a card
	- SyncIssueMessagePresenter: an OCMessagePresenter bridging messages to the legacy OCIssue interface
- ServerListTableViewController: provide option for auto resolution using a particular error
- Fixes
	- MediaUploadQueue: fix deadlock and make sure the tracking doesn't stop prematurely (also resulting in a deadlock)
	- ProgressSummarizer: fix dangling pointer warning

* - Update SDK to incorporate ocis OIDC change

* Various fixes:
- SwiftLint: add implicit_getter to disabled_rules as it is broken and produces false positive warnings
- MediaUploadQueue: fix deadlock and avoid premature end of tracking (resulting in a hung/dysfunctional app)
- ProgressSummarizer: fix warning
- FileProviderExtension: adopt new lock type API

* - Update SDK to include OCAuthenticationMethodOpenIDConnect changes for ocis

* - WebViewDisplayViewController: add support for content updates, clean up code
- PreviewViewController: add support for content updates, clean up code

* - ImageDisplayViewController: add support for image file updates
- MediaDisplayViewController: fix indentation
- PDFViewerViewController: add support for file updates, fix indentation

* - CardIssueMessagePresenter: option to only show one message at once

* - Update SDK for latest improvements

* - AppDelegate: register notification categories at launch
- ClientRootViewController: add and remove presenters at core start/stop
- RoundedLabel: clean up and add styling support via Style struct and switch all usages of RoundedLabel over to new syntax
- ServerListTableViewController
	- track message count on a per-bookmark basis
	- update app icon badge count
	- broadcast change notifications
- ServerListBookmarkCell: add message count badge
- DisplayHostViewController & QueryFileListTableViewController: stop observation of OCQuery.state before stopping OCQueries
- MessageSelector: make filtering optional to allow for usage without filtering
- NotificationManager
	- add support to create and register notification categories based on registered OCSyncIssueTemplates
	- add support for routing and handling notification responses via ComposeNotificationIdentifier() and NotificationResponseHandler category
- NotificationMessagePresenter: implement NotificationResponseHandler category to feed back notification responses to OCMessageQueue

* - ClientRootViewController: rename card message presenter var
- ClientSessionManager: new class to keep track of active client sessions and route messages for presentation. All ClientRootViewController instances should henceforth be created through it
- CardIssueMessagePresenter: add bookmark UUID to .identifier and adapt to new presentation completionHandler
- ServerListTableViewController
	- add support for ClientSessionManager delegation
	- add support for presenting messages after successful login
- SyncIssueMessagePresenter: add bookmark UUID to .identifier and adapt to new presentation completionHandler
- NotificationMessagePresenter
	- add bookmark UUID to .identifier and adapt to new presentation completionHandler
	- implement endPresentationOfMessage to remove notifications for removed messages
	- add support for taps on notifications

* - SDK update
- ClientSessionManager: change showMessage(notification:) and ClientSessionManagerDelegate from binary to based on priorities
- AlertView and AlertViewController: add optional header view and label
- CardIssueMessagePresenter: show bookmark .shortName as header for messages not belonging to the presenter's bookmark
- SceneDelegate and ThemeWindow: tracking which window is visible and in foreground
- ServerListTableViewController > ClientSessionManagerDelegate:
	- take foreground status into account when computing the presentation priority
	- add support for presenting messages from other accounts, based on foreground status and availability of "unused" scenes

* - Update SDK

* Add support for grouping messages:
- new MessageGroup class to store messages in groups
- MessageGroupCell replaces MessageCell
	- improve performance
	- add alternative layout for groups with more than one message:
		- switch to apply a choice to more than one message; badge count if switch is on
		- button to show all messages
- extend MessageSelector with support for maintaining MessageGroups
- MessageTableViewController to show messages as a list of individual messages

* - Update SDK

* - Update SDK

* - remove SyncIssueMessagePresenter as it is not used
- show messages only if they haven't already been resolved (previously shown until removed from the queue)
- update code to use OCMessage properties rather than OCMessage.syncIssue, replacing OCSyncIssue dependant code whereever possible
- add MessageQueueExample.swift to show how to use OCMessageQueue from within the app
- fix bug where messages without categoryIdentifier would be grouped together

* FileProvider changes:
- re-instate local error handling for creation of folders in the File Provider
- add support for class settings to disable local error handling for test purposes

* - bring up an alert if a new folder is attampted to be created in a location where another item with the same name already exists
- add support for providing a title to the namecheck/validation result in NamingViewController

* - OCMessage+Extension: convenience method to simplify presentation of OCMessages in the app
- move MessageSelector from ClientActivityViewController to ClientRootViewController for joint access from different parts of the app
- MessageSelector: add option to also collect and update a Set of OCSyncRecordIDs found in active messages
- ClientActivityCell + ClientActivityViewController: add support to show tappable warning sign instead of progress indicator if a message for the activity's sync record ID has been found
- ClientItemCell + FileListTableViewcontroller/ClientQueryViewController: add support to show tappable warning sign instead of progress indicator or more button if a message for the item's active sync record IDs has been found

* - Remove unused code from SDk

* - display "All done" message in Status tab when there are no messages or activities going on

* - Fix review findings

* - more nuanced status message in case of pending issues

* - Adding key command support to AlertViewController

* - remove duplicate/reunify diverged code in "ownCloudAppShared/SDK Extensions/OCItem+Extension.swift" and "ownCloud/SDK Extensions/OCItem+Extension.swift"
	- move app-specific code to OCItem+AppExtension.swift
	- update ownCloudAppShared/SDK Extensions/OCItem+Extension.swift with all changes
- optimize speed of:
	- OCItem+Extension.isShareRootItem() by checking only the parent item and cache the result for the duration of the runloop iteration
	- OCItem+Extension.shareRootItem(from:) by wrapping all SQLite lookups into a single transaction, avoiding unnecessary idle/wait time

* - Update SDK to add Cancel option for uploads

* - Add initial call to (re)set app icon count in case the app got deleted with messages but kept the count upon reinstall

* - Address finding (5) in #662 via SDK update ("ugly" authentication error message)

* - Extensions / Categories:
	- move OCBookmark.userName, .displayName and .shortName to ownCloudApp.framework so it becomes available to the NotificationMessagePresenter
- NotificationManager:
	- extend NotificationManager to provide equivalents for relevant methods of UNUserNotificationCenter
	- change all usages of UNUserNotificationCenter to NotificationManager
- NotificationMessagePresenter:
	- add account information to notifications
	- only add account information if the user has more than one account in the app
- Update SDK

* - Update SDK to address (6)

* - fix finding (6) glitch where message/warning icons wouldn't disappear when resuming a download or upload

* - Update SDK

* - Update SDK

Co-authored-by: Matthias Hühne <[email protected]>
Co-authored-by: Michael Neuwert <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>

* - Update SDK to develop (now containing all changes from feature/message-queue)

* - Fix static analyzer warnings

* - Update SDK to fix owncloud/enterprise#4053 and owncloud/ios-app#732

* - Update SDK

* [feature/branding] Branding / Static Login support (#637)

* - New Static Login
	- StaticLoginBundle: container for all Static Login parameters
	- StaticLoginProfile: individual parameters for a Static Login
	- StaticLoginViewController: WIP implementation for a Static Login UI
- Theme extensions
	- UIButton support
	- ThemeStyleIdentifier typealias for more expressive APIs
- Static Table View
	- added support for headerView and footerView of section

* - Add closeHandler-support to ClientRootViewController to run a block when a connection is closed by the user
- Fix crash in OCItem+Extension.lastModifiedInReadableFormat by adding a missing check
- Make sure ProgressHUDViewController.dismiss() always calls the completion block
- ServerListTableViewController enhancements:
    - hasToolbar-property to turn the toolbar on/off
    - make initialization with XIB optional
    - add showModal(), didUpdateServerList() and openBookmark() hooks for subclasses
- Add StaticLoginServerListViewController (ServerListTableViewController subclass)
- Add "Done" button to SettingsViewController if it is used as the rootViewController
- Add loginThemeStyleID to StaticLoginBundle
- Make StaticLoginSetupViewController feature-complete
- Add new ThemeView that can be used as container for ThemeAppliers that get removed automatically when the view is freed
- Relocate FullWidthHeaderView and make it a ThemeView subview
- Extend StaticTableViewSection.addButtonFooter() to support cancel-only versions
- Add toolbar to StaticLoginViewController
- Fix copyright notice formatting

* New Iconset for iOS: Filetype Icons

* Documentation for the app color scheme

* markdown definition for color theme values

* added links to type definitons

* added color definition

* added global colors

* added color values for dark scheme

* added light and classic color scheme

* Preparing branding support with themes and colors from plist file

* - using theme styles from plist file
- using splash image in welcome view

* set tint color for server list icon

* Add example images

* Change the color of the bookmark-logo

* Change colors to match example-theme

* use different icons

* All colors are not customizable by setting the color in Theme.plist. If the key is not set in plist, the default value will be used.

* fixed resolving theme color pair

* Added appiconset and changes branding colors

* Color adjustments and edited bookmark-logo

* Add Surf logos and colors

* setting new theme generic colors and reading them from keypath, if existing

* Add Surf logos and colors

* Add first SURF branding

* removed iOS 13 code

* Merge branch 'master' into Design

* fixed broken project file

* fixed broken plist file after merge

* Cleanup after cherry-pick from Design branch

* Add colors from SURF

* Add changes to Fastfile from tag 1.1.0

* Comment out Themes tests

* removed bookmark icon tint color

* Add new bookmark-icon.png

* Change the name to SURFdrive

* changed app name in build settings instead of target name

* customize search bar

* Add new bookmark icon and polish color settings

* Add new bookmark icon and polish color settings

* Change the name to SURFdrive, again

* - searchbar customization
- theming delete button in pass code view

* - Fix Swift and SwiftLint warnings
- Remove unused UploadsSettingsSection (was replaced by MediaUploadSettings)

* - adopted Fastfile to set the app name
- added app name to Themes.plist
- removed unneeded theme

* set correct path in Fastfile

* removed output name, because Bitrise expects other ipa name

* using branded image for quick access tab

* setting the placeholder color for UISearchBar text field in iOS 13

* check if app is branded, disable help and feedback, if branded app

* - support for branded urls (help, privacy)
- changed plist name

* hide Theme sett…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved by QA Approved by QA p3-medium Normal priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants