search

ClutchMederow / map-control

Map Control - Trading Platform
3 stars 0 forks source link

Bizon shows up as a mac10 in items #258

Closed drewproud closed 8 years ago

drenfr01 commented 8 years ago

so I investigated one of the market listings, the one for the usp-s image (name is bizon) for the weapon_aup. Looking at the items DB, it looks like the icon is just screwed up

db.items.findOne({_id: "iPMNoCvwk8zmQaEkb"})

iconURL: "https://steamcommunity-a.akamaihd.net/economy/image/-9a81dlWLwJ2UUGcVs_nsVtzdOEdtWwKGZZLQHTxDZ7I56KU0Zwwo4NUX4oFJZEHLbXH5ApeO4YmlhxYQknCRvCo04DEVlxkKgpoo6m1FBRp3_bGcjhQ09-jq5WYh8jkIbTWhG5C-8xnteXI8oThxg3lqBc5ZGHyd9KVIQ47YVqCqAPsx-i5hpHv75qfyCAy7CJz5H-OnxCpwUYbnuEcrvs"

drenfr01 commented 8 years ago

okay, I traced this for awhile, but couldn't see an obvious point. One interesting data point, the modified date is after the insert date on the item, which suggests it's been updated.

createdTimestamp: ISODate("2015-11-24T01:51:35.416Z"), modifiedTimestamp: ISODate("2015-11-24T03:49:49.255Z"),

drenfr01 commented 8 years ago

@drewproud actually, can you take a look at Db.js line 464? When you move items you reset the iconURL, potentially could that be an issue?

drewproud commented 8 years ago

This should be fixed now - the problem was using .each instead of .find when matching skin on deposit.