Various ticket: Remove deprecated API

src/Couchbase.Lite.Shared/API/Database/Collection.cs

@@ -91,9 +91,7 @@ public sealed unsafe class Collection : IChangeObservable<CollectionChangedEvent
         #region Properties
 
         // Must be called inside self lock
-#if !XAMARINIOS && !MONODROID
         [MemberNotNullWhen(false, nameof(c4Db), nameof(_c4coll))]
-#endif
         internal bool IsClosed => c4Db == null || _c4coll == null || !NativeSafe.c4coll_isValid(_c4coll);
 
         // Must be called inside self lock
@@ -948,9 +946,7 @@ internal bool IsIndexTrained(string name)
         /// <summary>
         /// Throws if this collection has been deleted, or its database closed.
         /// </summary>
-#if !XAMARINIOS && !MONOANDROID
         [MemberNotNull(nameof(c4Db), nameof(_c4coll))]
-#endif
         internal void CheckCollectionValid()
         {
             using var scope = ThreadSafety.BeginLockedScope();
@@ -987,7 +983,7 @@ internal void PostDatabaseChanged()
                 if (nChanges == 0 || external != newExternal || docIDs.Count > 1000) {
                     if (docIDs.Count > 0) {
                         // Only notify if there are actually changes to send
-                        var args = new CollectionChangedEventArgs(this, docIDs, Database);
+                        var args = new CollectionChangedEventArgs(this, docIDs);
                         _databaseChanged.Fire(this, args);
                         docIDs = new List<string>();
                     }

src/Couchbase.Lite.Shared/API/Database/CollectionChangedEventArgs.cs

@@ -16,6 +16,7 @@
 //  limitations under the License.
 //
 
+using System;
 using System.Collections.Generic;
 using System.Diagnostics;
 
@@ -24,9 +25,7 @@ namespace Couchbase.Lite
     /// <summary>
     /// The parameters of a collection changed event
     /// </summary>
-#pragma warning disable CS0618 // Type or member is obsolete
-    public sealed class CollectionChangedEventArgs : DatabaseChangedEventArgs
-#pragma warning restore CS0618 // Type or member is obsolete
+    public sealed class CollectionChangedEventArgs : EventArgs
     {
         #region Properties
 
@@ -35,14 +34,18 @@ public sealed class CollectionChangedEventArgs : DatabaseChangedEventArgs
         /// </summary>
         public Collection Collection { get; }
 
+        /// <summary>
+        /// Gets the document that was changed
+        /// </summary>
+        public IReadOnlyList<string> DocumentIDs { get; }
+
         #endregion
 
         #region Constructors
 
-        internal CollectionChangedEventArgs(Collection collection, IReadOnlyList<string> documentIDs,
-            Database database)
-            :base(database, documentIDs)
+        internal CollectionChangedEventArgs(Collection collection, IReadOnlyList<string> documentIDs)
         {
+            DocumentIDs = documentIDs; 
             Collection = collection;
         }
 

src/Couchbase.Lite.Shared/API/Database/DocumentChangedEventArgs.cs

@@ -33,12 +33,6 @@ public sealed class DocumentChangedEventArgs : EventArgs
         /// </summary>
         public string DocumentID { get; }
 
-        /// <summary>
-        /// [DEPRECATED] The source of the document that changed
-        /// </summary>
-        [Obsolete("Database is deprecated, please use Collection property instead")]
-        public Database Database { get; }
-
         /// <summary>
         /// The source of the document that changed
         /// </summary>
@@ -52,9 +46,6 @@ internal DocumentChangedEventArgs(string documentID, Collection collection)
         {
             DocumentID = documentID;
             Collection = collection;
-#pragma warning disable CS0618 // Type or member is obsolete
-            Database = collection.Database;
-#pragma warning restore CS0618 // Type or member is obsolete
         }
 
         #endregion

src/Couchbase.Lite.Shared/API/Database/Scope.cs

@@ -215,9 +215,7 @@ internal bool DeleteCollection(string name, string scope)
 
         #region Private Methods
 
-#if !XAMARINIOS && !MONODROID
         [MemberNotNull(nameof(c4Db))]
-#endif
         private void CheckOpen()
         {
             if (c4Db == null) {

src/Couchbase.Lite.Shared/API/Query/DataSource.cs

@@ -49,20 +49,6 @@ public static IDataSourceAs Collection(Collection collection)
             return new DatabaseSource(c, c.ThreadSafety);
         }
 
-        /// <summary>
-        /// [DEPRECATED] Creates a data source for an <see cref="IQuery" /> that gets results from the given
-        /// <see cref="Database" />
-        /// </summary>
-        /// <param name="database">The database to operate on</param>
-        /// <returns>The source of data for the <see cref="IQuery" /></returns>
-        [Obsolete("DataSource.Database is deprecated, please use DataSource.Collection")]
-        public static IDataSourceAs Database(Database database)
-        {
-            var db = CBDebug.MustNotBeNull(WriteLog.To.Query, Tag, nameof(database), database);
-
-            return new DatabaseSource(database, db.ThreadSafety);
-        }
-
         #endregion
     }
 }

src/Couchbase.Lite.Shared/API/Query/FullTextFunction.cs

@@ -27,27 +27,6 @@ namespace Couchbase.Lite.Query
     /// </summary>
     public static class FullTextFunction
     {
-        /// <summary>
-        /// Create a full-text matching function that will check for matches between given 
-        /// full text index name and the given query expression
-        /// </summary>
-        /// <param name="indexName">The name of the full-text index to perform the
-        /// check against</param>
-        /// <param name="query">The query expression to perform the check against</param>
-        /// <returns>A function that will perform the matching</returns>
-        [Obsolete("Use Match(index, query) instead")]
-        public static IExpression Match(string indexName, string query) =>
-            Match(Expression.FullTextIndex(indexName), query);
-
-        /// <summary>
-        /// Creates a full-text ranking value function indicating how well the current
-        /// query result matches the full-text query when performing the match comparison.
-        /// </summary>
-        /// <param name="indexName">The FTS index name to use when performing the calculation</param>
-        /// <returns>A function that will perform the ranking</returns>
-        [Obsolete("Use Rank(index) instead")]
-        public static IExpression Rank(string indexName) => Rank(Expression.FullTextIndex(indexName));
-
         /// <summary>
         /// Create a full-text matching function that will check for matches between given 
         /// full text index name and the given query expression

src/Couchbase.Lite.Shared/API/Query/IExpression.cs

@@ -116,14 +116,6 @@ public interface IExpression
         /// <returns>The expression representing the new operation</returns>
         IExpression IsNot(IExpression expression);
 
-        /// <summary>
-        /// [DEPRECATED] Gets an expression representing if the current expression is null
-        /// or missing (i.e. does not have a value)
-        /// </summary>
-        /// <returns>The expression representing the new operation</returns>
-        [Obsolete("This query expression deprecated, please use IsNotValued().")]
-        IExpression IsNullOrMissing();
-
         /// <summary>
         /// Gets an expression representing if the current expression has a value
         /// </summary>
@@ -184,14 +176,6 @@ public interface IExpression
         /// <returns>The expression representing the new operation</returns>
         IExpression NotEqualTo(IExpression expression);
 
-        /// <summary>
-        /// [DEPRECATED] Gets an expression representing if the current expression is neither null
-        /// nor missing (i.e. has a value)
-        /// </summary>
-        /// <returns>The expression representing the new operation</returns>
-        [Obsolete("This query expression deprecated, please use IsValued().")]
-        IExpression NotNullOrMissing();
-
         /// <summary>
         /// Logically "ors" the given expression with the current expression
         /// </summary>

src/Couchbase.Lite.Shared/API/Sync/Replicator+Backgrounding.cs

@@ -22,6 +22,7 @@
 using Foundation;
 using LiteCore.Interop;
 using System;
+using System.Linq;
 using System.Threading;
 using UIKit;
 
@@ -46,7 +47,7 @@ private NSFileProtection FileProtection
     {
         get {
 #pragma warning disable CS0618 // Type or member is obsolete
-            var attrs = NSFileManager.DefaultManager.GetAttributes(Config.Database.Path!);
+            var attrs = NSFileManager.DefaultManager.GetAttributes(Config.Collections.First().Database.Path!);
             return attrs?.ProtectionKey ?? NSFileProtection.None;
 #pragma warning restore CS0618 // Type or member is obsolete
         }

src/Couchbase.Lite.Shared/API/Sync/Replicator.cs

@@ -373,36 +373,6 @@ public void Stop()
             });
         }
 
-        /// <summary>
-        /// [DEPRECATED] Gets a list of document IDs that are going to be pushed, but have not been pushed yet
-        /// <item type="bullet">
-        /// <description>API is a snapshot and results may change between the time the call was made and the time</description>
-        /// </item>
-        /// </summary>
-        /// <returns>An immutable set of strings, each of which is a document ID</returns>
-        /// <exception cref="CouchbaseLiteException">Thrown if no push replication</exception>
-        /// <exception cref="CouchbaseException">Thrown if an error condition is returned from LiteCore</exception>
-        [Obsolete("GetPendingDocumentIDs() is deprecated, please use GetPendingDocumentIDs(Collection collection)")]
-        public IImmutableSet<string> GetPendingDocumentIDs()
-        {
-            return GetPendingDocumentIDs(Config.Database.GetDefaultCollection());
-        }
-
-        /// <summary>
-        /// [DEPRECATED] Checks whether or not a document with the given ID has any pending revisions to push
-        /// </summary>
-        /// <param name="documentID">The document ID</param>
-        /// <returns>A bool which represents whether or not the document with the corresponding ID has one or more pending revisions.  
-        /// <c>true</c> means that one or more revisions have not been pushed to the remote yet, 
-        /// and <c>false</c> means that all revisions on the document have been pushed</returns>
-        /// <exception cref="CouchbaseLiteException">Thrown if no push replication</exception>
-        /// <exception cref="CouchbaseException">Thrown if an error condition is returned from LiteCore</exception>
-        [Obsolete("IsDocumentPending(string documentID) is deprecated, please use IsDocumentPending(string documentID, Collection collection)")]
-        public bool IsDocumentPending(string documentID)
-        {
-            return IsDocumentPending(documentID, Config.Database.GetDefaultCollection());
-        }
-
         /// <summary>
         /// Checks whether or not a document with the given ID in the given collection is pending to push or not. 
         /// If the given collection is not part of the replication, an Invalid Parameter Exception will be thrown.