from cobra.core import Gene
from ssbio.core.protein import Protein
import os.path as op
import ssbio.utils
import cobra.io.dict
import logging
log = logging.getLogger(__name__)

class GenePro(Gene):
    """Extends the COBRAPy Gene object to add GEM-PRO annotations in the form of a Protein attribute
    """

    def __init__(self, id, name='', functional=True, root_dir=None, pdb_file_type='cif'):
        Gene.__init__(self, id=id, name=name, functional=functional)

        self.pdb_file_type = pdb_file_type

        # Create directories
        self._root_dir = None
        if root_dir:
            self.root_dir = root_dir

        self.protein = Protein(ident=id, root_dir=self.gene_dir, pdb_file_type=self.pdb_file_type)

    @property
    def root_dir(self):
        """Directory where Gene folder is located"""
        return self._root_dir

    @root_dir.setter
    def root_dir(self, path):
        if not path:
            raise ValueError('No path specified')

        if not op.exists(path):
            raise ValueError('{}: folder does not exist'.format(path))

        if self._root_dir:
            log.debug('Changing root directory of Gene "{}" from {} to {}'.format(self.id, self.root_dir, path))

            if not op.exists(op.join(path, self.id)):
                raise IOError('Gene "{}" does not exist in folder {}'.format(self.id, path))

        self._root_dir = path

        ssbio.utils.make_dir(self.gene_dir)

        # Propagate changes to protein
        if hasattr(self, 'protein'):
            self.protein.root_dir = self.gene_dir

    @property
    def gene_dir(self):
        """Gene folder"""
        if self.root_dir:
            return op.join(self.root_dir, self.id)
        else:
            return None

    def reset_protein(self):
        self.protein = Protein(self.id, root_dir=self.gene_dir, pdb_file_type=self.pdb_file_type)

    def __json_encode__(self):
        reqd_attribs = cobra.io.dict._REQUIRED_GENE_ATTRIBUTES
        optn_attribs = cobra.io.dict._OPTIONAL_GENE_ATTRIBUTES
        ordered_optn_attribs = cobra.io.dict._ORDERED_OPTIONAL_GENE_KEYS
        optn_attribs.update({'_root_dir': None})
        ordered_optn_attribs.append('_root_dir')
        new_gene = {key: str(getattr(self, key))
                    for key in reqd_attribs}
        cobra.io.dict._update_optional(self, new_gene, optn_attribs, ordered_optn_attribs)

        # Protein
        new_gene['protein'] = self.protein

        return new_gene

    def __json_decode__(self, **attrs):
        for k, v in attrs.items():
            setattr(self, k, v)

@@ -2526,35 +2526,115 @@
     
-for k, v in attrs.items():%0A
+Gene.__init__(self, id=attrs%5B'id'%5D)%0A        for k, v in attrs.items():%0A            if k not in %5B'id'%5D:%0A    
     

Fix JSON decode