fix review

Author: adelowo

models/migrations/v96.go

@@ -39,5 +39,9 @@ func deleteOrphanedAttachments(x *xorm.Engine) error {
 				_, err := x.ID(attachment.ID).NoAutoCondition().Delete(attachment)
 				return err
 			})
-	return err
+	if err != nil {
+		return err
+	}
+
+	return sess.Commit()
 }

modules/setting/repository.go

@@ -100,11 +100,11 @@ var (
 		AccessControlAllowOrigin:                "",
 		UseCompatSSHURI:                         false,
 		DefaultCloseIssuesViaCommitsInAnyBranch: false,
+		EnablePushCreateUser:                    false,
+		EnablePushCreateOrg:                     false,
 		EnableKanbanBoard:                       true,
 		ProjectBoardBasicKanbanType:             strings.Split("Todo, In progress, Done", ","),
 		ProjectBoardBugTriageType:               strings.Split("Needs Triage, High priority, Low priority, Closed", ","),
-		EnablePushCreateUser:                    false,
-		EnablePushCreateOrg:                     false,
 
 		// Repository editor settings
 		Editor: struct {