Skip to content

Commit fb9d04e

Browse files
author
sreeder
committed
fix bug when accesing existing sourc
1 parent fb61fb3 commit fb9d04e

File tree

4 files changed

+10
-6
lines changed

4 files changed

+10
-6
lines changed

odmtools/controller/pageSource.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,8 @@ def __init__(self, parent, title, service_manager, src):
4444
self.panel.lstSource.SetStringItem(num_items, 8, str(s.state))
4545
self.panel.lstSource.SetStringItem(num_items, 9, str(s.zip_code))
4646
self.panel.lstSource.SetStringItem(num_items, 10, str(s.citation))
47+
self.panel.lstSource.SetStringItem(num_items, 11, str(s.id))
48+
4749

4850
if s.organization == src.organization:
4951
index = i
@@ -101,9 +103,9 @@ def getSource(self):
101103
s = self.prev_val
102104
elif self.rbSelect.Value:
103105
index = self.lstSource.GetFirstSelected()
104-
code = self.lstSource.GetItem(index, 0).GetText()
106+
code = self.lstSource.GetItem(index, 11).GetText()
105107
logger.debug(code)
106-
s = self.series_service.get_source_by_org(code)
108+
s = self.series_service.get_src_by_id(code)
107109
elif self.rbCreate.Value:
108110
s = self.createdSrc
109111
return s
@@ -122,6 +124,8 @@ def show_new_src(self, src):
122124
self.txtNewSrc.SetStringItem(0, 9, str(src.citation))
123125
self.txtNewSrc.SetStringItem(0, 10, str(src.iso_metadata_id))
124126

127+
128+
125129
self.txtNewSrc.Focus(0)
126130
self.txtNewSrc.Select(0)
127131
self.txtNewSrc.Enable(True)

odmtools/gui/wizSave.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -392,13 +392,13 @@ def on_wizard_finished(self, event):
392392
else:
393393
Method = self.record_service.create_method(Method)
394394

395-
s= self.series_service.source_exists(Source)
395+
s= self.record_service.get_src(Source)
396396
if s:
397397
if Source == self.currSeries.source:
398398
Source = None
399399
else:
400400
# Source = self.record_service.get_source(Source)
401-
Source = source
401+
Source = s
402402
else:
403403
Source = self.record_service.create_source(Source)
404404

odmtools/odmservices/series_service.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,6 @@ def get_db_version(self):
4646
def get_all_sources(self):
4747
return self._edit_session.query(Source).order_by(Source.id).all()
4848

49-
def get_source_by_org(self, org):
50-
return self._edit_session.query(Source).filter(Source.organization == org).first()
5149

5250
def get_src_by_id(self, src_id):
5351
try:

odmtools/view/clsWizSource.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,10 @@ def __init__(self,prnt,id,pos,size,style,name):
4343
self.lstSource.InsertColumn(8, 'State')
4444
self.lstSource.InsertColumn(9, 'ZipCode')
4545
self.lstSource.InsertColumn(10, 'Citation')
46+
self.lstSource.InsertColumn(11, 'ID')
4647
self.lstSource.SetColumnWidth(0, 100)
4748
self.lstSource.SetColumnWidth(1, 100)
49+
self.lstSource.SetColumnWidth(11, 0)
4850

4951
self.lstSource.Bind(wx.EVT_LIST_ITEM_SELECTED,
5052
self.OnListCtrl1ListItemSelected)

0 commit comments

Comments
 (0)