Merge PR MacDownApp#1360: restore Open Recent menu

Author: macnotes

MacDown/Code/Document/MPDocument.m

@@ -546,7 +546,14 @@ - (BOOL)writeToURL:(NSURL *)url ofType:(NSString *)typeName
             self.editor.selectedRange = selection;
         }
     }
-    return [super writeToURL:url ofType:typeName error:outError];
+    BOOL ok = [super writeToURL:url ofType:typeName error:outError];
+    if (ok)
+    {
+        NSDocumentController *controller =
+            [NSDocumentController sharedDocumentController];
+        [controller mp_noteNewRecentDocumentURL:url];
+    }
+    return ok;
 }
 
 - (NSData *)dataOfType:(NSString *)typeName error:(NSError **)outError
@@ -564,6 +571,11 @@ - (BOOL)readFromData:(NSData *)data ofType:(NSString *)typeName
 
     self.loadedString = content;
     [self reloadFromLoadedString];
+
+    // Ensure the file is registered for the system "Open Recent" menu.
+    NSDocumentController *controller =
+        [NSDocumentController sharedDocumentController];
+    [controller mp_noteNewRecentDocumentURL:self.fileURL];
     return YES;
 }
 

MacDown/Code/Extension/NSDocumentController+Document.h

@@ -13,4 +13,9 @@
 - (__kindof NSDocument *)createNewEmptyDocumentForURL:(NSURL *)url
         display:(BOOL)display error:(NSError * __autoreleasing *)error;
 
+// Explicitly note a recently used document URL for the system "Open Recent"
+// menu. We filter out temporary files to avoid polluting the list with app
+// internals (help/treat files, etc.).
+- (void)mp_noteNewRecentDocumentURL:(NSURL *)url;
+
 @end

MacDown/Code/Extension/NSDocumentController+Document.m

@@ -10,6 +10,36 @@
 
 @implementation NSDocumentController (Document)
 
+- (void)mp_noteNewRecentDocumentURL:(NSURL *)url
+{
+    if (!url || !url.isFileURL)
+        return;
+
+    NSString *path = url.path;
+    if (!path.length)
+        return;
+
+    // Skip temp files (help, treats, other internal/generated docs).
+    NSString *tmp = NSTemporaryDirectory();
+    if (tmp.length)
+    {
+        NSString *tmpResolved =
+            [[tmp stringByStandardizingPath] stringByResolvingSymlinksInPath];
+        if (tmpResolved.length && ![tmpResolved hasSuffix:@"/"])
+            tmpResolved = [tmpResolved stringByAppendingString:@"/"];
+
+        NSString *pathResolved =
+            [[path stringByStandardizingPath] stringByResolvingSymlinksInPath];
+        if (tmpResolved.length && [pathResolved hasPrefix:tmpResolved])
+            return;
+    }
+
+    // Recent-documents updates are UI-adjacent; keep it on the main queue.
+    dispatch_async(dispatch_get_main_queue(), ^{
+        [self noteNewRecentDocumentURL:url];
+    });
+}
+
 - (__kindof NSDocument *)createNewEmptyDocumentForURL:(NSURL *)url
         display:(BOOL)display error:(NSError * __autoreleasing *)error
 {
@@ -24,6 +54,7 @@ - (__kindof NSDocument *)createNewEmptyDocumentForURL:(NSURL *)url
 
     doc.draft = YES;
     doc.fileURL = url;
+    [self mp_noteNewRecentDocumentURL:url];
     return doc;
 }