eve-seat / seat

SeAT 0x. [UNSUPPORTED]
https://github.com/eveseat/seat
MIT License
69 stars 38 forks source link

[Bug] CustomsOffice update: Trying to get property of non-object #312

Closed smokie128 closed 9 years ago

smokie128 commented 9 years ago

Stacktrace: http://pastebin.com/p3FqN7xg

eve-seat commented 9 years ago

Ping @LunarchildEU could you have a look at this one?

ghost commented 9 years ago

Please execute the following querys in your DB: select itemID, locationID from corporation_assetlist where itemID = 1012674593736;

locationID is needed for next Query: select count(*) from mapDenormalize where mapDenormalize.groupID = 7 and mapDenormalize.solarSystemID = <locationID>;

That should yield at least the rowcount for the certain System, if not please execute select count(*) from mapDenormalize;.

These querys all should return somthing > 0.

smokie128 commented 9 years ago
mysql> select itemID, locationID from corporation_assetlist where itemID = 1012674593736;
+---------------+------------+
| itemID        | locationID |
+---------------+------------+
| 1012674593736 |   30003054 |
+---------------+------------+
1 row in set (0.01 sec)

mysql> select count(*) from mapDenormalize where mapDenormalize.groupID = 7 and mapDenormalize.solarSystemID = 30003054;
+----------+
| count(*) |
+----------+
|       10 |
+----------+
1 row in set (0.00 sec)
ghost commented 9 years ago

Currently the function that pops this error relies that the POCO is already in the AssetList table of the Corporation, which sadly turns out it is not good.

That could have caused the problem - 9a14350cd4437502fdfc7c772171a53fd7eddb7e should sort that out.

eve-seat commented 9 years ago

Sweet. @smokie128 can you test @LunarchildEU fix and let us know?

smokie128 commented 9 years ago

Tested #313 Working fine so far. Issued a full update on the corp key, finished without errors.