photo

type Collection interface {
	GetName() CollectionName
	//Description
	GetDescription() (string, error)
	SetDescription(string) error
	//Photos (or links to them via id)
	LinkPhoto(PhotoHash) error
	UnlinkPhoto(PhotoHash) error
	GetAllPhotoHashes() ([]PhotoHash, error)
}

type CollectionName string

type Image interface {
	GetFormat() imageformat.Format
	GetSize() uint64
	GetResolution() resolution.Resolution
	GetPhoto() Photo
	GetData() (io.ReadCloser, error)
	SetData(io.Reader) error
}

type Photo interface {
	GetHash() PhotoHash
	GetDescription() (string, error)
	SetDescription(string) error
	GetTimeTaken() (time.Time, error)
	SetTimeTaken(time.Time) error
	GetImages() ([]Image, error)
	GetImageByResolutionID(resolution.ResolutionID) (Image, error)

	//****** I am not sure I like this. Would rather have a "NewPhoto func that generated internally *********"
	//Can only be set once if trying to overwrite should fail
	SetOriginalImage(fmt imageformat.Format, size uint64, height, width uint64, data io.Reader) (Image, error)
	//Can only be set once if trying to overwrite should fail
	SetMasterImage(fmt imageformat.Format, size uint64, height, width uint64, data io.Reader) (Image, error)

	//Should check to see if we have an image of that resolution first
	//Commenting out as I think that we shouldn't have so setable fields
	// NewImage(fmt imageformat.Format, size uint64, height, width uint64, data io.Reader) (Image, error)
	CreateSubImage(resolution.Resolution) error
	RemoveImage(resolution.ResolutionID) error
}

type PhotoHash string