Skip to content

Conversation

@vanlooverenkoen
Copy link
Owner

Description

Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.

Related Issues

Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
  • All existing and new tests are passing.
  • I updated/added relevant documentation (doc comments with ///).
  • The analyzer (flutter analyze) does not report any problems on my PR.
  • I read and followed the Flutter Style Guide.
  • The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
  • I updated CHANGELOG.md to add a description of the change.
  • I signed the CLA.
  • I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

  • Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
  • No, this is not a breaking change.

ditman and others added 30 commits November 18, 2020 11:42
* Render markers fromBytes. Ensure initial icon is also preserved. Add test.
* Opt-out tests from null-safety until plugin is migrated.
* Android Code Inspection and Clean up

* Android Code Inspection and Clean up

* Android Code Inspection and Clean up

* Android Code Inspection and Clean up

* Android Code Inspection and Clean up

* Android Code Inspection and Clean up

* Import PluginRegistry

* Import PluginRegistry

* Update GoogleMapController.java

* google_maps_flutter

* Update build.gradle

* Update Convert.java

* Update Convert.java

Add TODO comment

* Update Convert.java
* Make sure only camera_platform_interface has updates

* add dev dependency to async package

* refactored import to package import

* Fix formatting issues
* Remove unchecked  && deprecated Warnings

* Remove settings_aar.gradle

* Update CHANGELOG.md

* Fix Format

* Fix Format

* espresso

* Format

* SuppressWarnings
…3313)

* Expanded platform interface so support setting flash mode

* Formatted dart code

* Manually serialize flash mode enum rather than relying on stringification.

* Add default to flash mode serialization
* Suppress unchecked cast warning in java test

* Bumped version number
With NNBD assert(foo != null) for a non nullable foo generates an analysis error.
However as long as we support mixed-mode we want these asserts it, so disabling the check.
* Add zoom support to platform interface

* Added method to retrieve min supported zoom level

* Bumped version to 1.0.2

* Fixed small typo
* code sample fixed

* updated CHANGELOG.md and pubspec.yaml
Update Flutter SDK constraint to match templates.
Update Flutter SDK constraint to match templates.
Update Flutter SDK constraint to match templates.
Update Flutter SDK constraint to match templates.
…#3302)

* First suggestion for camera platform interface

* Remove test coverage folder

* Renamed onLatestImageAvailableHandler definition

* Split CameraEvents into separate streams

* Implemented & tested first parts of method channel implementation

* Remove unused EventChannelMock class

* Add missing unit tests

* Added placeholders in default method channel implementation

* Updated platform interface

* Update packages/camera/camera_platform_interface/lib/src/platform_interface/camera_platform.dart

Co-authored-by: Maurits van Beusekom <[email protected]>

* Add unit test for availableCameras

* Expand availableCameras unit test. Added unit test for takePicture.

* Add unit test for startVideoRecording

* Add unit test for prepareForVideoRecording

* Add unit test for stopVideoRecording

* Add unit test for pauseVideoRecording

* Add unit test for buildView

* WIP: Dart and Android implementation

* Fix formatting

* Have resolution stream replay last value on subscription. Replace stream_transform with rxdart.

* Added reverse method channel to replace event channel. Updated initialise and takePicture implementations for android. WIP implementation for startVideoRecording

* Fixed example app for Android. Removed isRecordingVideo and isStreamingImages from buildView method.

* Added some first tests for camera/camera

* More tests and some feedback

* iOS implementation: Removed standard event channel. Added reverse method channel. Updated initialize method. Added resolution changed event. Updated error reporting to use new method channel.

* Started splitting initialize method

* Finish splitting up initialize for iOS

* Update unit tests

* Fix takePicture method on iOS

* Split initialize method on Android

* Fix video recording on iOS. Updated platform interface.

* Update unit tests

* Update error handling of video methods in iOS code. Make iOS code more consistent.

* Updated startVideoRecording documentation

* Make sure file is returned by stopVideoRecording

* Use correct event-type after initializing

* Fix DartMessenger unit-tests

* Change cast

* Fix formatting

* Fixed tests, formatting and analysis warnings

* Added missing license to Dart files

* Updated CHANGELOG and version

* Added additional unit-tests to platform_interface

* Added more tests

* Formatted code

* Re-added the CameraPreview widget

* Use import/export instead of part implementation

* fixed formatting

* Resolved additional feedback

* Update dependency to git repo

* Depend on pub.dev for camera_platform_interface

* Fix JAVA formatting

* Fix changelog

* Make sure camera package can be published

* Assert when stream methods are called from wrong platform

* Add dev_dependency on plugin_platform_interface package, required by tests.

* Remove pedantic requirement from initialize() method. Remove unnecessary completers.

* Remove dependency on dart:io

* Restrict exposed types from platform interface

* Moved test for image stream in separate file

* Fixed formatting issue

* Fix deprecation warning

* Apply feedback from bparrishMines

* Fix formatting issues

* Removed redundant podspec files

* Removed redundant ios files

* Handle SecurityException

Co-authored-by: Maurits van Beusekom <[email protected]>
Co-authored-by: Maurits van Beusekom <[email protected]>
Co-authored-by: daniel <[email protected]>
Co-authored-by: David Iglesias Teixeira <[email protected]>
mvanbeusekom and others added 21 commits December 16, 2020 08:45
…um (#3326)

* Fix formatting issues

* Make sure torch value is serialized correctly
* updated README.md to notify about no < iOS10 support

* Version bump

* Improved explanation

Co-authored-by: Maurits van Beusekom <[email protected]>

Co-authored-by: Maurits van Beusekom <[email protected]>
…elled (#3303)

Set networkCallback back to null when event stream is cancelled
* Move camera to camera/camera

* Fix relative path after move

* First suggestion for camera platform interface

* Remove test coverage folder

* Update the version to 1.0.0

* Remove redundant analysis overrides

* Renamed onLatestImageAvailableHandler definition

* Split CameraEvents into separate streams

* Updated platform interface to have recording methods return XFile instances.

* Update documentation and unit tests to match platform interface changes

* Make file input optional for recording methods in platform interface. Update docs.

* Add missing full stop in docs.

* Run dartfmt. Wrapped docs after max 80 cols. Added missing full stop.

* Implemented & tested first parts of method channel implementation

* Remove unused EventChannelMock class

* Add missing unit tests

* Add availableCameras to method channel implementation

* Updated platform interface

* Update packages/camera/camera_platform_interface/lib/src/platform_interface/camera_platform.dart

Co-authored-by: Maurits van Beusekom <[email protected]>

* Update packages/camera/camera_platform_interface/lib/src/platform_interface/camera_platform.dart

Co-authored-by: Maurits van Beusekom <[email protected]>

* Added placeholders in default method channel implementation

* Add missing implementations to default method channel implementation

* Fix formatting

* Fix PR feedback

* Add unit test for availableCameras

* Expand availableCameras unit test. Added unit test for takePicture.

* Add unit test for startVideoRecording

* Add unit test for prepareForVideoRecording

* Add unit test for stopVideoRecording

* Add unit test for pauseVideoRecording

* Add unit test for buildView

* Remove TODO comment

* Update packages/camera/camera_platform_interface/lib/src/method_channel/method_channel_camera.dart

Co-authored-by: Maurits van Beusekom <[email protected]>

* Update packages/camera/camera_platform_interface/lib/src/method_channel/method_channel_camera.dart

Co-authored-by: Maurits van Beusekom <[email protected]>

* Update packages/camera/camera_platform_interface/lib/src/method_channel/method_channel_camera.dart

Co-authored-by: Maurits van Beusekom <[email protected]>

* WIP: Dart and Android implementation

* Have resolution stream replay last value on subscription. Replace stream_transform with rxdart.

* Added reverse method channel to replace event channel. Updated initialise and takePicture implementations for android. WIP implementation for startVideoRecording

* Fixed example app for Android. Removed isRecordingVideo and isStreamingImages from buildView method.

* iOS implementation: Removed standard event channel. Added reverse method channel. Updated initialize method. Added resolution changed event. Updated error reporting to use new method channel.

* Added some first tests for camera/camera

* Started splitting initialize method

* More tests and some feedback

* Finish splitting up initialize for iOS

* Update unit tests

* Remove unused listener in plugin

* Fix takePicture method on iOS

* Fix video recording on iOS. Updated platform interface.

* Update unit tests

* Update error handling of video methods in iOS code. Make iOS code more consistent.

* Split initialize method on Android

* Updated startVideoRecording documentation

* Make sure file is returned by stopVideoRecording

* Change cast

* Use correct event-type after initializing

* Fix DartMessenger unit-tests

* Fix formatting

* Fixed tests, formatting and analysis warnings

* Added missing documentation public APIs

* Added missing license to Dart files

* Fix formatting issues

* Updated CHANGELOG and version

* Added more tests

* Formatted code

* Added additional unit-tests to platform_interface

* Fix formatting issues

* Re-added the CameraPreview widget

* Refactored CameraException not to use

* Use import/export instead of part implementation

* fixed formatting

* Resolved additional feedback

* Resolved additional feedback

* Flash WIP

* Implement flash modes for Android

* Update dependency to git repo

* Add missing PictureCaptureRequest class

* Add PR feedback

* Move enums out of Camera.java

* Expanded platform interface so support setting flash mode

* Formatted dart code

* Manually serialize flash mode enum rather than relying on stringification.

* Add default to flash mode serialization

* Fix for unit tests and reformatting

* Fixed CHANGELOG and remove redundant iOS files

* Expose FlashMode through camera package

* Add reqeusted unit tests

* Clean up new tests

* Add unit tests for Android implementation

* Update platform interface dependency to point to pub.dev release.

Co-authored-by: Maurits van Beusekom <[email protected]>
Co-authored-by: Maurits van Beusekom <[email protected]>
Co-authored-by: daniel <[email protected]>
* Refactored and tested zoom on Android

* Fix merge conflict
* Added torch mode functionality for Android and iOS.

* Format objective c code
* Wait pre capture to finish

* Add autofocus stage to capture

* Fixed autofocus stage

* Make sure torch is turned off

* Format & structure improvements

* Update changelog and pubspec

* Update packages/camera/camera/android/src/main/java/io/flutter/plugins/camera/Camera.java

Co-authored-by: Maurits van Beusekom <[email protected]>

* Update packages/camera/camera/android/src/main/java/io/flutter/plugins/camera/PictureCaptureRequest.java

Co-authored-by: Maurits van Beusekom <[email protected]>

* Update packages/camera/camera/CHANGELOG.md

Co-authored-by: Maurits van Beusekom <[email protected]>

* Update packages/camera/camera/pubspec.yaml

Co-authored-by: Maurits van Beusekom <[email protected]>

* Remove unnecessary import.

* Updated unit tests

Co-authored-by: Maurits van Beusekom <[email protected]>
Co-authored-by: Maurits van Beusekom <[email protected]>
@vanlooverenkoen vanlooverenkoen merged commit bfccbd6 into vanlooverenkoen:master Dec 27, 2020
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.