Add links for local and remote urls to context menu (#320)

* Add links for local and remote urls to context menu

* Make community and user apId fields nullable

* Show a home and fediverse icon in links

Author: jwr1

assets/icons/fediverse.png

@@ -133,6 +133,7 @@ abstract class CommentModel with _$CommentModel {
     required bool? canAuthUserModerate,
     required NotificationControlStatus? notificationControlStatus,
     required List<String>? bookmarks,
+    required String? apId,
   }) = _CommentModel;
 
   factory CommentModel.fromMbin(JsonMap json) => CommentModel(
@@ -163,6 +164,7 @@ abstract class CommentModel with _$CommentModel {
     canAuthUserModerate: json['canAuthUserModerate'] as bool?,
     notificationControlStatus: null,
     bookmarks: optionalStringList(json['bookmarks']),
+    apId: json['apId'] as String?,
   );
 
   factory CommentModel.fromLemmy(
@@ -230,6 +232,7 @@ abstract class CommentModel with _$CommentModel {
         // Empty string indicates comment is saved. No string indicates comment is not saved.
         if (json['saved'] as bool) '',
       ],
+      apId: lemmyComment['ap_id'] as String,
     );
   }
 
@@ -324,6 +327,7 @@ abstract class CommentModel with _$CommentModel {
         // Empty string indicates comment is saved. No string indicates comment is not saved.
         if (json['saved'] as bool) '',
       ],
+      apId: piefedComment['ap_id'] as String,
     );
   }
 }

lib/src/models/comment.dart

@@ -63,6 +63,7 @@ abstract class DetailedCommunityModel with _$DetailedCommunityModel {
     required bool? isBlockedByUser,
     required bool isPostingRestrictedToMods,
     required NotificationControlStatus? notificationControlStatus,
+    required String? apId,
   }) = _DetailedCommunityModel;
 
   factory DetailedCommunityModel.fromMbin(JsonMap json) {
@@ -93,6 +94,7 @@ abstract class DetailedCommunityModel with _$DetailedCommunityModel {
           : NotificationControlStatus.fromJson(
               json['notificationStatus'] as String,
             ),
+      apId: json['apProfileId'] as String?,
     );
 
     communityMentionCache[community.name] = community;
@@ -123,6 +125,7 @@ abstract class DetailedCommunityModel with _$DetailedCommunityModel {
       isPostingRestrictedToMods:
           (lemmyCommunity['posting_restricted_to_mods']) as bool,
       notificationControlStatus: null,
+      apId: lemmyCommunity['actor_id'] as String,
     );
 
     communityMentionCache[community.name] = community;
@@ -170,6 +173,7 @@ abstract class DetailedCommunityModel with _$DetailedCommunityModel {
           : communityView['activity_alert'] as bool
           ? NotificationControlStatus.loud
           : NotificationControlStatus.default_,
+      apId: piefedCommunity['actor_id'] as String,
     );
 
     communityMentionCache[community.name] = community;
@@ -184,24 +188,37 @@ abstract class CommunityModel with _$CommunityModel {
     required int id,
     required String name,
     required ImageModel? icon,
+    required String? apId,
   }) = _CommunityModel;
 
   factory CommunityModel.fromMbin(JsonMap json) => CommunityModel(
     id: json['magazineId'] as int,
     name: json['name'] as String,
     icon: mbinGetOptionalImage(json['icon'] as JsonMap?),
+    apId: json['apProfileId'] as String?,
   );
 
   factory CommunityModel.fromLemmy(JsonMap json) => CommunityModel(
     id: json['id'] as int,
     name: getLemmyPiefedActorName(json),
     icon: lemmyGetOptionalImage(json['icon'] as String?),
+    apId: json['actor_id'] as String,
   );
 
   factory CommunityModel.fromPiefed(JsonMap json) => CommunityModel(
     id: json['id'] as int,
     name: getLemmyPiefedActorName(json),
     icon: lemmyGetOptionalImage(json['icon'] as String?),
+    apId: json['actor_id'] as String,
+  );
+
+  factory CommunityModel.fromDetailedCommunity(
+    DetailedCommunityModel community,
+  ) => CommunityModel(
+    id: community.id,
+    name: community.name,
+    icon: community.icon,
+    apId: community.apId,
   );
 }
 

lib/src/models/community.dart

@@ -54,6 +54,7 @@ abstract class DetailedUserModel with _$DetailedUserModel {
     required bool? isFollowerOfUser,
     required bool? isBlockedByUser,
     required NotificationControlStatus? notificationControlStatus,
+    required String? apId,
   }) = _DetailedUserModel;
 
   factory DetailedUserModel.fromMbin(JsonMap json) {
@@ -75,6 +76,7 @@ abstract class DetailedUserModel with _$DetailedUserModel {
           : NotificationControlStatus.fromJson(
               json['notificationStatus'] as String,
             ),
+      apId: json['apProfileId'] as String?,
     );
 
     userMentionCache[user.name] = user;
@@ -99,6 +101,7 @@ abstract class DetailedUserModel with _$DetailedUserModel {
       isFollowerOfUser: null,
       isBlockedByUser: (json['blocked'] as bool?) ?? false,
       notificationControlStatus: null,
+      apId: lemmyPerson['actor_id'] as String,
     );
   }
 
@@ -123,6 +126,7 @@ abstract class DetailedUserModel with _$DetailedUserModel {
           : json['activity_alert'] as bool
           ? NotificationControlStatus.loud
           : NotificationControlStatus.default_,
+      apId: piefedPerson['actor_id'] as String,
     );
   }
 }
@@ -135,6 +139,7 @@ abstract class UserModel with _$UserModel {
     required ImageModel? avatar,
     required DateTime? createdAt,
     required bool isBot,
+    required String? apId,
   }) = _UserModel;
 
   factory UserModel.fromMbin(JsonMap json) => UserModel(
@@ -143,6 +148,7 @@ abstract class UserModel with _$UserModel {
     avatar: mbinGetOptionalImage(json['avatar'] as JsonMap?),
     createdAt: optionalDateTime(json['createdAt'] as String?),
     isBot: (json['isBot'] ?? false) as bool,
+    apId: json['apProfileId'] as String?,
   );
 
   factory UserModel.fromLemmy(JsonMap json) => UserModel(
@@ -151,6 +157,7 @@ abstract class UserModel with _$UserModel {
     avatar: lemmyGetOptionalImage(json['avatar'] as String?),
     createdAt: DateTime.parse(json['published'] as String),
     isBot: json['bot_account'] as bool,
+    apId: json['actor_id'] as String,
   );
 
   factory UserModel.fromPiefed(JsonMap json) => UserModel(
@@ -159,6 +166,7 @@ abstract class UserModel with _$UserModel {
     avatar: lemmyGetOptionalImage(json['avatar'] as String?),
     createdAt: DateTime.parse(json['published'] as String),
     isBot: json['bot'] as bool,
+    apId: json['actor_id'] as String,
   );
 
   factory UserModel.fromDetailedUser(DetailedUserModel user) => UserModel(
@@ -167,6 +175,7 @@ abstract class UserModel with _$UserModel {
     avatar: user.avatar,
     createdAt: user.createdAt,
     isBot: user.isBot,
+    apId: user.apId,
   );
 }
 

lib/src/models/user.dart

@@ -7,6 +7,7 @@ import 'package:interstellar/src/models/community.dart';
 import 'package:interstellar/src/models/post.dart';
 import 'package:interstellar/src/screens/explore/community_owner_panel.dart';
 import 'package:interstellar/src/screens/explore/community_screen.dart';
+import 'package:interstellar/src/utils/ap_urls.dart';
 import 'package:interstellar/src/utils/language.dart';
 import 'package:interstellar/src/utils/utils.dart';
 import 'package:interstellar/src/widgets/image_selector.dart';
@@ -74,7 +75,7 @@ class _CreateScreenState extends State<CreateScreen> {
       }
 
       String body = 'Cross posted from ';
-      body += post.apId ?? genPostUrl(context, post).toString();
+      body += genPostUrls(context, post).last.toString();
       if (post.body != null && post.body!.trim().isNotEmpty) {
         body += '\n\n';
         // Wrap original body with markdown quote

lib/src/screens/feed/create_screen.dart

@@ -6,8 +6,8 @@ import 'package:interstellar/src/api/notifications.dart';
 import 'package:interstellar/src/controller/controller.dart';
 import 'package:interstellar/src/controller/server.dart';
 import 'package:interstellar/src/models/comment.dart';
-import 'package:interstellar/src/models/post.dart';
 import 'package:interstellar/src/screens/feed/post_comment_screen.dart';
+import 'package:interstellar/src/utils/ap_urls.dart';
 import 'package:interstellar/src/utils/utils.dart';
 import 'package:interstellar/src/widgets/ban_dialog.dart';
 import 'package:interstellar/src/widgets/content_item/content_item.dart';
@@ -283,7 +283,6 @@ class _PostCommentState extends State<PostComment> {
                 community: widget.comment.community,
               );
             },
-      openLinkUri: genCommentUrl(context, widget.comment),
       editDraftResourceId:
           'edit:${widget.comment.postType.name}:comment:${context.watch<AppController>().instanceHost}:${widget.comment.id}',
       replyDraftResourceId:
@@ -354,6 +353,7 @@ class _PostCommentState extends State<PostComment> {
               );
             },
       onClick: widget.onClick ?? collapse,
+      shareLinks: genCommentUrls(context, widget.comment),
     );
 
     final menuWidget = IconButton(

lib/src/screens/feed/post_comment.dart

@@ -5,6 +5,7 @@ import 'package:interstellar/src/api/notifications.dart';
 import 'package:interstellar/src/controller/controller.dart';
 import 'package:interstellar/src/controller/server.dart';
 import 'package:interstellar/src/models/post.dart';
+import 'package:interstellar/src/utils/ap_urls.dart';
 import 'package:interstellar/src/utils/utils.dart';
 import 'package:interstellar/src/widgets/ban_dialog.dart';
 import 'package:interstellar/src/widgets/content_item/content_item.dart';
@@ -241,7 +242,6 @@ class _PostItemState extends State<PostItem> {
                   );
                 },
           numComments: widget.item.numComments,
-          openLinkUri: genPostUrl(context, widget.item),
           editDraftResourceId:
               'edit:${widget.item.type.name}:${ac.instanceHost}:${widget.item.id}',
           replyDraftResourceId:
@@ -322,6 +322,7 @@ class _PostItemState extends State<PostItem> {
           isCompact: widget.isCompact,
           onClick: widget.isTopLevel ? widget.onTap : null,
           crossPost: widget.item,
+          shareLinks: genPostUrls(context, widget.item),
         ),
       ),
     );

lib/src/screens/feed/post_item.dart

@@ -6,6 +6,7 @@ import 'package:interstellar/src/models/comment.dart';
 import 'package:interstellar/src/models/post.dart';
 import 'package:interstellar/src/screens/feed/post_comment.dart';
 import 'package:interstellar/src/screens/feed/post_item.dart';
+import 'package:interstellar/src/utils/ap_urls.dart';
 import 'package:interstellar/src/utils/utils.dart';
 import 'package:interstellar/src/widgets/menus/content_menu.dart';
 import 'package:interstellar/src/widgets/context_menu.dart';
@@ -217,7 +218,6 @@ class _PostPageState extends State<PostPage> {
               );
             },
       numComments: crossPost.numComments,
-      openLinkUri: genPostUrl(context, crossPost),
       editDraftResourceId:
           'edit:${crossPost.type.name}:${ac.instanceHost}:${crossPost.id}',
       replyDraftResourceId:
@@ -300,6 +300,7 @@ class _PostPageState extends State<PostPage> {
               );
             },
       crossPost: crossPost,
+      shareLinks: genPostUrls(context, crossPost),
     );
     showContentMenu(context, contentItem);
   }

lib/src/screens/feed/post_page.dart

@@ -0,0 +1,92 @@
+import 'package:flutter/material.dart';
+import 'package:interstellar/src/controller/controller.dart';
+import 'package:interstellar/src/controller/server.dart';
+import 'package:interstellar/src/models/comment.dart';
+import 'package:interstellar/src/models/community.dart';
+import 'package:interstellar/src/models/post.dart';
+import 'package:interstellar/src/models/user.dart';
+import 'package:interstellar/src/utils/utils.dart';
+import 'package:provider/provider.dart';
+
+/// Local url first, remote url last.
+/// Only one link if local is the source.
+List<Uri> genPostUrls(BuildContext context, PostModel post) {
+  final ac = context.read<AppController>();
+
+  final apUrl = post.apId == null ? null : Uri.tryParse(post.apId!);
+
+  return [
+    if (apUrl == null || apUrl.host != ac.instanceHost)
+      Uri.https(
+        ac.instanceHost,
+        ac.serverSoftware == ServerSoftware.mbin
+            ? '/m/${post.community.name}/${switch (post.type) {
+                PostType.thread => 't',
+                PostType.microblog => 'p',
+              }}/${post.id}'
+            : '/post/${post.id}',
+      ),
+    ?apUrl,
+  ];
+}
+
+/// Local url first, remote url last.
+/// Only one link if local is the source.
+List<Uri> genCommentUrls(BuildContext context, CommentModel comment) {
+  final ac = context.read<AppController>();
+
+  final apUrl = comment.apId == null ? null : Uri.tryParse(comment.apId!);
+
+  return [
+    if (apUrl == null || apUrl.host != ac.instanceHost)
+      Uri.https(
+        ac.instanceHost,
+        ac.serverSoftware == ServerSoftware.mbin
+            ? '/m/${comment.community.name}/${switch (comment.postType) {
+                PostType.thread => 't',
+                PostType.microblog => 'p',
+              }}/${comment.postId}/-/${switch (comment.postType) {
+                PostType.thread => 'comment',
+                PostType.microblog => 'reply',
+              }}/${comment.id}'
+            : '/comment/${comment.id}',
+      ),
+    ?apUrl,
+  ];
+}
+
+/// Local url first, remote url last.
+/// Only one link if local is the source.
+List<Uri> genCommunityUrls(BuildContext context, CommunityModel community) {
+  final ac = context.read<AppController>();
+
+  final apUrl = community.apId == null ? null : Uri.tryParse(community.apId!);
+
+  return [
+    if (apUrl == null || apUrl.host != ac.instanceHost)
+      Uri.https(
+        ac.instanceHost,
+        ac.serverSoftware == ServerSoftware.mbin
+            ? '/m/${community.name}'
+            : '/c/${community.name}',
+      ),
+    ?apUrl,
+  ];
+}
+
+/// Local url first, remote url last.
+/// Only one link if local is the source.
+List<Uri> genUserUrls(BuildContext context, UserModel user) {
+  final ac = context.read<AppController>();
+
+  final apUrl = user.apId == null ? null : Uri.tryParse(user.apId!);
+
+  return [
+    if (apUrl == null || apUrl.host != ac.instanceHost)
+      Uri.https(
+        ac.instanceHost,
+        '/u/${ac.serverSoftware == ServerSoftware.mbin && getNameHost(context, user.name) != ac.instanceHost ? '@' : ''}${user.name}',
+      ),
+    ?apUrl,
+  ];
+}