review notes

Author: xster

packages/share/android/build.gradle

@@ -1,4 +1,4 @@
-group 'com.yourcompany.share'
+group 'io.flutter.plugins.share'
 version '1.0-SNAPSHOT'
 
 buildscript {

packages/share/android/src/main/java/io/flutter/plugins/share/SharePlugin.java

@@ -38,16 +38,19 @@ public void onMethodCall(MethodCall call, MethodChannel.Result result) {
         return;
       }
       final String text = (String) call.arguments;
-
-      Intent shareIntent = new Intent();
-      shareIntent.setAction(Intent.ACTION_SEND);
-      shareIntent.putExtra(Intent.EXTRA_TEXT, text);
-      shareIntent.setType("text/plain");
-      context.startActivity(Intent.createChooser(shareIntent, null /* dialog title optional */));
+      share(text);
       result.success(null);
     } else {
       result.error("UNKNOWN_METHOD", "Unknown share method called", null);
     }
   }
 
+  private void share(String text) {
+    Intent shareIntent = new Intent();
+    shareIntent.setAction(Intent.ACTION_SEND);
+    shareIntent.putExtra(Intent.EXTRA_TEXT, text);
+    shareIntent.setType("text/plain");
+    context.startActivity(Intent.createChooser(shareIntent, null /* dialog title optional */));
+  }
+
 }

packages/share/example/android/app/src/main/AndroidManifest.xml

@@ -1,5 +1,5 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
-    package="com.yourcompany.share_example"
+    package="io.flutter.plugins.share_example"
     android:versionCode="1"
     android:versionName="0.0.1">
 

packages/share/example/android/app/src/main/java/com/yourcompany/share_example/MainActivity.java

@@ -1,3 +1,7 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
 package com.yourcompany.share_example;
 
 import android.os.Bundle;

packages/share/example/ios/Runner/AppDelegate.h

@@ -1,3 +1,7 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
 #import <UIKit/UIKit.h>
 #import <Flutter/Flutter.h>
 

packages/share/example/ios/Runner/AppDelegate.m

@@ -1,3 +1,7 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
 #include "AppDelegate.h"
 #include "PluginRegistry.h"
 

packages/share/example/ios/Runner/main.m

@@ -1,3 +1,7 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
 #import <UIKit/UIKit.h>
 #import <Flutter/Flutter.h>
 #import "AppDelegate.h"

packages/share/ios/Classes/SharePlugin.m

@@ -17,12 +17,7 @@ - (instancetype)initWithController:
   [shareChannel setMethodCallHandler:^(FlutterMethodCall *call,
                                        FlutterResult result) {
     if ([@"share" isEqualToString:call.method]) {
-      UIActivityViewController *activityViewController =
-          [[UIActivityViewController alloc] initWithActivityItems:@[call.arguments]
-                                            applicationActivities:nil];
-      [controller presentViewController:activityViewController
-                               animated:YES
-                             completion:nil];
+      [self share:call.arguments];
       result(nil);
     } else {
       result([FlutterError errorWithCode:@"UNKNOWN_METHOD"
@@ -32,4 +27,14 @@ - (instancetype)initWithController:
   }];
 }
 
+
+- (void)share:(id)sharedItems {
+  UIActivityViewController *activityViewController =
+      [[UIActivityViewController alloc] initWithActivityItems:@[sharedItems]
+                                        applicationActivities:nil];
+  [controller presentViewController:activityViewController
+                            animated:YES
+                          completion:nil];
+}
+
 @end