sqlite3.OperationalError: ambiguous column name: Item.fdev_id when exporting about trade-dangerous HOT 7 CLOSED

On Fri, Jan 4, 2019, 07:35 Peter Magnusson ***@***.*** wrote: I'm proposing the following for Item table CREATE TABLE Item ( item_id INTEGER PRIMARY KEY, name VARCHAR(40) COLLATE nocase, category_id INTEGER NOT NULL, ui_order INTEGER NOT NULL DEFAULT 0, avg_price INTEGER, fdev_id INTEGER, UNIQUE(item_id), FOREIGN KEY (category_id) REFERENCES Category(category_id) ON UPDATE CASCADE ON DELETE CASCADE ); CREATE INDEX idx_item_by_fdev_id ON Item (fdev_id); CREATE INDEX idx_item_by_item_id_fdev_id ON Item (item_id, fdev_id); — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#35 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABpz4DRSlQr83QyxoIaaE_UnymOL7UCtks5u_2augaJpZM4ZptdZ> .

On Fri, Jan 4, 2019, 08:44 Jonathan Jones ***@***.*** wrote: StationItem is for database, internal use only and should never be exported. If you want the market data, that what the prices file is for. I haven't been looking at what you've been doing because I've been spending all my time fixing my computer, but I will at some point. On Fri, Jan 4, 2019, 07:35 Peter Magnusson ***@***.*** wrote: > I'm proposing the following for Item table > > CREATE TABLE Item > ( > item_id INTEGER PRIMARY KEY, > name VARCHAR(40) COLLATE nocase, > category_id INTEGER NOT NULL, > ui_order INTEGER NOT NULL DEFAULT 0, > avg_price INTEGER, > fdev_id INTEGER, > > UNIQUE(item_id), > > FOREIGN KEY (category_id) REFERENCES Category(category_id) > ON UPDATE CASCADE > ON DELETE CASCADE > ); > CREATE INDEX idx_item_by_fdev_id ON Item (fdev_id); > CREATE INDEX idx_item_by_item_id_fdev_id ON Item (item_id, fdev_id); > > — > You are receiving this because you are subscribed to this thread. > Reply to this email directly, view it on GitHub > <#35 (comment)>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/ABpz4DRSlQr83QyxoIaaE_UnymOL7UCtks5u_2augaJpZM4ZptdZ> > . >

On Fri, Jan 4, 2019, 10:50 Peter Magnusson ***@***.*** wrote: I noticed this while writing unit tests. What I am doing to trigger it is trade export --all-tables --path="c:\temp\export\" and it fails with the sqlite3.OperationalError: ambiguous column name: Item.fdev_id message. The option --all-tables should export prices according to the help texts and it tries to do it using CSV. In csvexport.py, which is used by the export command, it includes the StationItem table. One way of fixing this is to exclude StationItem from all csv export but then the export_cmd.py file needs to be changed as well to reflect this. But in that case there would be a need for another way of exporting the prices. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#35 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABpz4BQ25yfI8lE-ppAltvVxbgyyKFkUks5u_5RMgaJpZM4ZptdZ> .

On Fri, Jan 4, 2019, 17:07 Jonathan Jones ***@***.*** wrote: The fix is simply to separate the two fields into two separate UNIQUE lines. On Fri, Jan 4, 2019, 10:50 Peter Magnusson ***@***.*** wrote: > I noticed this while writing unit tests. > What I am doing to trigger it is trade export --all-tables > --path="c:\temp\export\" and it fails with the sqlite3.OperationalError: > ambiguous column name: Item.fdev_id message. > > The option --all-tables should export prices according to the help texts > and it tries to do it using CSV. > In csvexport.py, which is used by the export command, it includes the > StationItem table. > > One way of fixing this is to exclude StationItem from all csv export but > then the export_cmd.py file needs to be changed as well to reflect this. > But in that case there would be a need for another way of exporting the > prices. > > — > You are receiving this because you commented. > Reply to this email directly, view it on GitHub > <#35 (comment)>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/ABpz4BQ25yfI8lE-ppAltvVxbgyyKFkUks5u_5RMgaJpZM4ZptdZ> > . >