From 6c41719a020e16a33fcb06a14d6a4e1e5571feb6 Mon Sep 17 00:00:00 2001
From: Richard Ipsum <richard.ipsum@codethink.co.uk>
Date: Wed, 19 Aug 2015 17:14:04 +0000
Subject: Simplify accessors in package

Change-Id: I38dc72c2348883abea56bd58a3195df4dc197c8a
---
 baserockimport/package.py | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

(limited to 'baserockimport/package.py')

diff --git a/baserockimport/package.py b/baserockimport/package.py
index 361022f..6360cd8 100644
--- a/baserockimport/package.py
+++ b/baserockimport/package.py
@@ -23,9 +23,9 @@ class Package(object):
 
     '''
     def __init__(self, kind, name, version):
-        self.kind = kind
-        self.name = name
-        self.version = version
+        self._kind = kind
+        self._name = name
+        self._version = version
         self.required_by = []
         self.morphology = None
         self.repo_url = None
@@ -57,18 +57,18 @@ class Package(object):
                 self.name == name and
                 self.version == version)
 
-    # FIXME: these accessors are useless, but I want there to be some way
-    # of making it clear that some of the state of the Package object is
-    # mutable and some of the state is not ...
+    @property
+    def kind(self):
+        return self._kind
 
-    def set_morphology(self, morphology):
-        self.morphology = morphology
+    @property
+    def name(self):
+        return self._name
 
-    def set_dependencies(self, dependencies):
-        self.dependencies = dependencies
+    @property
+    def version(self):
+        return self._version
 
-    def set_is_build_dep(self, is_build_dep):
-        self.is_build_dep = is_build_dep
-
-    def set_version_in_use(self, version_in_use):
-        self.version_in_use = version_in_use
+    @property
+    def parent(self):
+        return self.required_by[0] if len(self.required_by) > 0 else None
-- 
cgit v1.2.1