few fixes

src/Api/Startup.cs

@@ -80,6 +80,7 @@ namespace Bit.Api
             services.AddSingleton<IOrganizationUserRepository, SqlServerRepos.OrganizationUserRepository>();
             services.AddSingleton<ISubvaultRepository, SqlServerRepos.SubvaultRepository>();
             services.AddSingleton<ISubvaultUserRepository, SqlServerRepos.SubvaultUserRepository>();
+            services.AddSingleton<IFolderRepository, SqlServerRepos.FolderRepository>();
 
             // Context
             services.AddScoped<CurrentContext>();

src/Sql/dbo/Stored Procedures/Cipher_Create.sql

@@ -5,7 +5,10 @@
     @Type TINYINT,
     @Data NVARCHAR(MAX),
     @CreationDate DATETIME2(7),
-    @RevisionDate DATETIME2(7)
+    @RevisionDate DATETIME2(7),
+    -- Extra, unused props from cipher details model since dapper maps all child properties too. This is kind of a hack.
+    @FolderId UNIQUEIDENTIFIER,
+    @Favorite BIT
 AS
 BEGIN
     SET NOCOUNT ON

src/Sql/dbo/Stored Procedures/Cipher_Update.sql

@@ -5,7 +5,10 @@
     @Type TINYINT,
     @Data NVARCHAR(MAX),
     @CreationDate DATETIME2(7),
-    @RevisionDate DATETIME2(7)
+    @RevisionDate DATETIME2(7),
+    -- Extra, unused props from cipher details model since dapper maps all child properties too. This is kind of a hack.
+    @FolderId UNIQUEIDENTIFIER,
+    @Favorite BIT
 AS
 BEGIN
     SET NOCOUNT ON