Corrected/added formatting and comments.

Author: msallin

SQLite.CodeFirst/Convention/SqliteForeignKeyIndexConvention.cs

@@ -9,6 +9,12 @@
 
 namespace SQLite.CodeFirst.Convention
 {
+    /// <summary>
+    /// Renames the indicies, generated by the 
+    /// <see cref="ForeignKeyIndexConvention"/>-Convention, to match the scheme: "IX_{TableName}_{PropertyName}".
+    /// This is necessary because in SQLite an index-name must be unique.
+    /// Must be added right after the <see cref="ForeignKeyIndexConvention"/>-Convention.
+    /// </summary>
     public class SqliteForeignKeyIndexConvention : IStoreModelConvention<AssociationType>
     {
         private const string IndexAnnotationName = "http://schemas.microsoft.com/ado/2013/11/edm/customannotation:Index";
@@ -34,13 +40,15 @@ public virtual void Apply(AssociationType item, DbModel model)
                 var annotation = GetAnnotation(edmProperty.MetadataProperties, IndexAnnotationName);
                 if (annotation != null)
                 {
-                    // The original attribute is removed. The noneForeignKeyIndicies will be remained and readded without any modification
+                    // The original attribute is removed. The none-ForeignKeyIndicies will be remained and readded without any modification
                     // and the foreignKeyIncidies will be readded with the correct name.
                     edmProperty.RemoveAnnotation(IndexAnnotationName);
 
+                    // The schema for the automatically generated index name is "IX_{TableName}_{PropertyName}"
                     var noneForeignKeyIndicies = annotation.Indexes.Where(index => index.Name != "IX_" + edmProperty.Name);
                     IndexAnnotation newIndexAnnotation = new IndexAnnotation(noneForeignKeyIndicies);
 
+                    // The schema for a FK index, which is generated by the Entity Framework, is "IX_{PropertyName}"
                     var foreignKeyIndicies = annotation.Indexes.Where(index => index.Name == "IX_" + edmProperty.Name);
                     foreach (var foreignKeyIndex in foreignKeyIndicies)
                     {
@@ -54,7 +62,6 @@ public virtual void Apply(AssociationType item, DbModel model)
             }
         }
 
-
         private static IndexAnnotation GetAnnotation(IEnumerable<MetadataProperty> metadataProperties, string name)
         {
             foreach (MetadataProperty metadataProperty in metadataProperties)

SQLite.CodeFirst/SqliteInitializerBase.cs

@@ -1,7 +1,7 @@
 using System;
 using System.Data.Entity;
 using System.Data.Entity.ModelConfiguration.Conventions;
-  using SQLite.CodeFirst.Convention;
+using SQLite.CodeFirst.Convention;
 
 namespace SQLite.CodeFirst
 {
@@ -17,6 +17,9 @@ protected SqliteInitializerBase(DbModelBuilder modelBuilder)
             // This convention will crash the SQLite Provider before "InitializeDatabase" gets called.
             // See https://github.com/msallin/SQLiteCodeFirst/issues/7 for details.
             modelBuilder.Conventions.Remove<TimestampAttributeConvention>();
+
+            // Place the own ForeinKeyIndexConvention right after the original.
+            // The own convention will rename the automatically created indicies by using the correct scheme.
             modelBuilder.Conventions.AddAfter<ForeignKeyIndexConvention>(new SqliteForeignKeyIndexConvention());
         }