Ticket #32 (closed: open)

Opened 17 years ago

Last modified 17 years ago

AddEcospaceHabitat method Return iNewPos is zero indexed..Should be one indexed.??

Reported by: user8 Owned by: administrator
Priority: normal Milestone: Ecopath 6: build 6.0.7
Component: Database / Import Version:
Severity: minor Keywords:


For consistency, I think it should be one indexed..Like the way core uses one based index method while returning EcospaceHabitats?.. The core did the offset by 1 in its module..

So I think while this database method returns iNewPos, it should be offset by 1 before giving that value to UI? Make sense?

Jeroen, could you also check AddEcospaceMPA, and AddEcospaceRegion?? seems these two already offset by 1..from UI behavior..

Change History

comment:1 Changed 17 years ago by jeroens

AddEcospaceHabitat?, AddEcospaceMPA and AddEcospaceRegion? now correctly return the Index of the item rather than their core array position.

Note: See TracTickets for help on using tickets.