Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Inventory Glitch #33

Closed
ParzivalGG opened this issue Sep 25, 2021 · 12 comments · Fixed by #131
Closed

Inventory Glitch #33

ParzivalGG opened this issue Sep 25, 2021 · 12 comments · Fixed by #131
Labels
bug Something isn't working

Comments

@ParzivalGG
Copy link

If somebody is buying custom weapon models, then it works for that particular period and afterwards when they join again their inventory is gone, and they've to buy it again.
@nuclearsilo583
Copy link
Owner

nuclearsilo583 commented Sep 25, 2021 •
edited
Loading

If you using mysql as database. Can you give me a screenshot at the time player buy an item in the store_plugins_log table.
If you're using sqlite open the .qc3 file and give me the same table screenshot

@ParzivalGG
Copy link
Author

image
There ya go!

@ParzivalGG
Copy link
Author

image

@nuclearsilo583
Copy link
Owner

Could keep using it for a couple of day to see if this may happen again.

@nuclearsilo583
Copy link
Owner

Close due to inactive

@KillerRoi
Copy link

It is still not saving

@KillerRoi
Copy link

Everytime, players need to join and equip it again and again

@Oz-Lin
Copy link

Oz-Lin commented Aug 23, 2022

Sorry for necroing.
This happens when the previous playerskin was expired but not removed from store_equipment database table.
Everytime when a player reports the store bug to me, I have to remove the loadout from the database manually.

L 08/23/2022 - 02:23:20: [store.smx] SQL error happened.
Query: INSERT INTO store_equipment (`player_id`, `type`, `unique_id`, `slot`) VALUES(438, "playerskin", "uid_model_megumin", 2)
Error: Duplicate entry '438-playerskin-2' for key 'PRIMARY'

@Oz-Lin Oz-Lin mentioned this issue Aug 23, 2022
Open
@nuclearsilo583 nuclearsilo583 added the bug Something isn't working label Aug 23, 2022
@nuclearsilo583
Copy link
Owner

Can you give me screenshot me the store_equipment table structure.

@Oz-Lin
Copy link

Oz-Lin commented Aug 24, 2022

Here you go.
图片

@nuclearsilo583
Copy link
Owner

probaly you have done something to that table because the table doesn't contain any primary keys and unique key which make the data can't insert duplicate. This is how the structure from myside even with the fresh/ currently created table. You should backup the store_equipment table and drop it via mysql. And restart the server so it can create that table again with the most exact sync.
image

@Oz-Lin
Copy link

Oz-Lin commented Aug 24, 2022

Thanks, hopefully the problem is solved after moving values from the old store_equipment to the new one.

nuclearsilo583 added a commit that referenced this issue Aug 24, 2022
@nuclearsilo583
update core compatible with latest chat-processor version, update inc…
2e31dd7 
…lude files, fix inventory glitch on store_equipment table (#33, #102)
azalty added a commit to azalty/zephyrus-store-preview-new-syntax that referenced this issue Dec 20, 2022
@azalty
Fixed items being removed in a certain scenario nuclearsilo583#33
56c3aad 
Items would get removed from a player's inventory if they removed it then got it again in the same map.
This bug affected every store item.
@azalty azalty mentioned this issue Dec 20, 2022
Merged
nuclearsilo583 pushed a commit that referenced this issue Dec 28, 2022
@azalty @nuclearsilo583
Fixed items being removed in a certain scenario #33
2b365b6 
Items would get removed from a player's inventory if they removed it then got it again in the same map.
This bug affected every store item.
HotoCocoaco added a commit to HotoCocoaco/zephyrus-store-preview-new-syntax that referenced this issue Feb 3, 2023
@HotoCocoaco
Squashed commit of the following:
fd29f94 
commit d816215
Author: nuclear silo <[email protected]>
Date:   Fri Feb 3 14:05:56 2023 +0700

    oops

commit 6c93deb
Author: nuclear silo <[email protected]>
Date:   Fri Feb 3 03:56:57 2023 +0700

    various bugs fixes/ improvement. Please check changelogs.txt

commit 8c120cf
Author: azalty <[email protected]>
Date:   Mon Jan 23 15:56:02 2023 +0100

    Fixed load queries going through map changes

commit 652123c
Author: azalty <[email protected]>
Date:   Mon Jan 23 15:33:19 2023 +0100

    Fixed a rare error when selling an item during map change

commit 67a15a0
Author: azalty <[email protected]>
Date:   Mon Jan 23 15:28:24 2023 +0100

    Changed handling of restricted items nuclearsilo583#103

    Changed handling of restricted items when players don't have access to them anymore nuclearsilo583#103

    Added the "sm_store_sell_restricted" cvar and prevented players from equipping restricted items

commit a93fa66
Author: azalty <[email protected]>
Date:   Mon Jan 23 13:40:24 2023 +0100

    Fixed problems with Native_SetClientCredits

commit 447614f
Author: Kroytz <[email protected]>
Date:   Sun Jan 8 19:08:49 2023 +0800

    Add cvar controller

commit b7d450e
Author: Kroytz <[email protected]>
Date:   Sun Jan 8 14:06:41 2023 +0800

    Fix permanent item override by ticked item

commit 3d968c5
Author: Kroytz <[email protected]>
Date:   Sun Jan 8 14:04:30 2023 +0800

    Extend item instead override item.

commit 6d36ca9
Author: azalty <[email protected]>
Date:   Mon Dec 26 18:33:32 2022 +0100

    Fixed gifts store module ignoring sm_store_gifts_enabled

    Caused problems with other plugins using gifts

commit 2b365b6
Author: azalty <[email protected]>
Date:   Wed Dec 21 00:42:12 2022 +0100

    Fixed items being removed in a certain scenario nuclearsilo583#33

    Items would get removed from a player's inventory if they removed it then got it again in the same map.
    This bug affected every store item.

commit bbe230c
Author: azalty <[email protected]>
Date:   Tue Dec 20 17:59:55 2022 +0100

    Fixed SQL injection exploits, SQL syntax improvements

commit af60593
Author: AiDN™ <[email protected]>
Date:   Sat Oct 29 14:59:19 2022 +0200

    remove compiling error - g_cvarChatTag

commit b0b4c96
Author: nuclear silo <[email protected]>
Date:   Sat Oct 29 18:15:21 2022 +0700

    mistaken

commit 6edea08
Author: nuclear silo <[email protected]>
Date:   Sat Oct 29 16:41:58 2022 +0700

    add center tag cvar to fix giveaway center tag display (tks Koen for reporting)

commit f6d26f5
Author: nuclear silo <[email protected]>
Date:   Sat Oct 29 00:59:28 2022 +0700

    Update store_combine.sp

commit 3826c91
Author: ezyyeah <[email protected]>
Date:   Wed Oct 26 01:15:21 2022 +0200

    crowns abuse fix 3

commit a349a4d
Author: ezyyeah <[email protected]>
Date:   Wed Oct 26 01:02:16 2022 +0200

    fixing client credits can be lower than 0

commit a399bc0
Author: Erik-97 <[email protected]>
Date:   Sun Oct 23 19:45:09 2022 +0200

    crowns abuse fix 2

commit 3212a1c
Author: Erik-97 <[email protected]>
Date:   Sat Oct 22 20:17:18 2022 +0200

    crowns abuse fix

    https://streamable.com/gl561j

commit 2ecd5c8
Author: azalty <[email protected]>
Date:   Sat Oct 22 15:04:39 2022 +0200

    Fixed nuclearsilo583#118

    We should check if the client is valid first, before checking if they're alive

commit bf5f87a
Author: azalty <[email protected]>
Date:   Sun Sep 4 18:43:59 2022 +0200

    CheckSteamAuth: sSteam[8] instead of sSteam[7]

    Allows the "steam" "1:57298004" syntax instead of "steam" "STEAM_1:1:57298004"

commit c68ee4a
Author: azalty <[email protected]>
Date:   Sun Sep 4 18:33:02 2022 +0200

    Made CheckSteamAuth ignore the steamid2 universe

commit 68d17e1
Author: exd02 <[email protected]>
Date:   Thu Sep 1 22:14:48 2022 -0300

    Added pt-BR translation and remove duplicated-keys

commit b69a5f8
Author: nuclear silo <[email protected]>
Date:   Sun Aug 28 12:25:44 2022 +0700

    re-add combined core version, added client reset loadout command (nuclearsilo583#102), add create item voucher by uid for root admin

commit 7470954
Author: AiDN™ <[email protected]>
Date:   Fri Aug 26 17:49:01 2022 +0200

    Update store-trade.sp

commit 2ac4b81
Author: AiDN™ <[email protected]>
Date:   Fri Aug 26 16:52:30 2022 +0200

    fix store-trade and store.phrases

    store-trade.sp edited the Array index out-of-bounds (index -1, limit 128) --> removed Chat, using colorvariables CPrintToChat
    added } to 2 translations: QuizzGenerated and Hol placed

    store-trade tested: ❌ (now testing on my server to see if it works)

commit 4c7ae40
Merge: fd9b55e 23969f0
Author: nuclear silo <[email protected]>
Date:   Sat Aug 27 14:40:17 2022 +0700

    Merge pull request nuclearsilo583#106 from nuclearsilo583/beta

    seperate core plugin to sub include file for easy view/edit

commit fd9b55e
Author: nuclear silo <[email protected]>
Date:   Fri Aug 26 00:15:09 2022 +0700

    wrong date

commit 2ff5cf0
Author: nuclear silo <[email protected]>
Date:   Fri Aug 26 00:10:24 2022 +0700

    add bonus system on most of gambling modules, change daily check in method from 24hrs to server date (nuclearsilo583#98)

commit 23969f0
Author: nuclear silo <[email protected]>
Date:   Thu Aug 25 17:12:59 2022 +0700

    seperate core plugin to sub include file for easy view/edit

commit 6cec12b
Author: nuclear silo <[email protected]>
Date:   Thu Aug 25 16:50:51 2022 +0700

    fix vip item can be equipped with expired vip users (nuclearsilo583#103)

commit 2e31dd7
Author: nuclear silo <[email protected]>
Date:   Wed Aug 24 17:46:31 2022 +0700

    update core compatible with latest chat-processor version, update include files, fix inventory glitch on store_equipment table (nuclearsilo583#33, nuclearsilo583#102)

commit 2ac8d60
Author: nuclear silo <[email protected]>
Date:   Tue Aug 23 00:11:23 2022 +0700

    fix trail translation, remove/update resource link for lower git size

commit 9844729
Author: nuclear silo <[email protected]>
Date:   Sat Aug 6 19:43:51 2022 +0700

    Update changelog.txt

commit a881e02
Author: nuclear silo <[email protected]>
Date:   Sat Aug 6 19:39:01 2022 +0700

    fix utf8 format for special character symbols

commit c29ea28
Author: azalty <[email protected]>
Date:   Fri Aug 5 23:36:35 2022 +0200

    Small improvements to OnConfigExecuted

commit 5776364
Author: nuclear silo <[email protected]>
Date:   Sat Aug 6 01:10:54 2022 +0700

    Fix core database character set with special character

commit 28c5949
Author: azalty <[email protected]>
Date:   Fri Aug 5 19:52:03 2022 +0200

    Quick fix

commit dffc3a0
Author: azalty <[email protected]>
Date:   Fri Aug 5 19:25:36 2022 +0200

    Fixed a crash related to model precaching

    Also added the repo url

commit 92e0f90
Author: nuclear silo <[email protected]>
Date:   Fri Aug 5 01:33:01 2022 +0700

    Fix sprays item can be used for free (nuclearsilo583#92)

commit fe6da15
Author: nuclear silo <[email protected]>
Date:   Tue Aug 2 01:42:24 2022 +0700

    fix earning's knife kill exploit

commit c2aec05
Author: Bernardino Sousa <[email protected]>
Date:   Fri Jul 29 00:11:47 2022 +0100

    Fix multiple database connections

commit b7830df
Author: nuclear silo <[email protected]>
Date:   Sun Jul 24 22:34:40 2022 +0700

    beta of create item voucher option for root admin

commit 938575f
Author: nuclear silo <[email protected]>
Date:   Sun Jul 24 18:59:36 2022 +0700

    Update README.md

commit 3273d9c
Author: nuclear silo <[email protected]>
Date:   Sat Jul 23 23:16:12 2022 +0700

    Major fixes with SM 1.11 stack size (nuclearsilo583#87), fix translation file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed items being removed in a certain scenario #33 azalty/zephyrus-store-preview-new-syntax
4 participants
@Oz-Lin @KillerRoi @nuclearsilo583 @ParzivalGG