From e81f86f942ad281746bba5d259b217bbef690d3f Mon Sep 17 00:00:00 2001
From: Andrey Smirnov <Smirnov.Andrey@gmail.com>
Date: Mon, 21 Apr 2014 21:26:44 +0400
Subject: [PATCH] Slight refactoring, add oldRefs. #8

---
 deb/publish.go | 26 ++++++++++++--------------
 1 file changed, 12 insertions(+), 14 deletions(-)

diff --git a/deb/publish.go b/deb/publish.go
index 4521063d..c512937b 100644
--- a/deb/publish.go
+++ b/deb/publish.go
@@ -34,9 +34,14 @@ type PublishedRepo struct {
 	// SourceUUID is UUID of either snapshot or local repo
 	SourceUUID string `codec:"SnapshotUUID"`
 
-	snapshot    *Snapshot
-	localRepo   *LocalRepo
+	// Pointer to snapshot if SourceKind == "snapshot"
+	snapshot *Snapshot
+	// Pointer to local repo if SourceKind == "local"
+	localRepo *LocalRepo
+	// Package references is SourceKind == "local"
 	packageRefs *PackageRefList
+	// Old (currently created) package references
+	oldRefs *PackageRefList
 }
 
 // NewPublishedRepo creates new published repository
@@ -203,7 +208,10 @@ func (p *PublishedRepo) RefList() *PackageRefList {
 	if p.SourceKind == "local" {
 		return p.packageRefs
 	}
-	return p.snapshot.RefList()
+	if p.SourceKind == "snapshot" {
+		return p.snapshot.RefList()
+	}
+	panic("unknown source")
 }
 
 // Encode does msgpack encoding of PublishedRepo
@@ -249,18 +257,8 @@ func (p *PublishedRepo) Publish(packagePool aptly.PackagePool, publishedStorage
 		progress.Printf("Loading packages...\n")
 	}
 
-	var refList *PackageRefList
-
-	if p.snapshot != nil {
-		refList = p.snapshot.RefList()
-	} else if p.localRepo != nil {
-		refList = p.localRepo.RefList()
-	} else {
-		panic("no source")
-	}
-
 	// Load all packages
-	list, err := NewPackageListFromRefList(refList, collectionFactory.PackageCollection(), progress)
+	list, err := NewPackageListFromRefList(p.RefList(), collectionFactory.PackageCollection(), progress)
 	if err != nil {
 		return fmt.Errorf("unable to load packages: %s", err)
 	}