[UploadUtilities] Fix 403 IQDB error when checking duplicates of local file upload #382
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Currently, the duplicate check doesn't fire when a file is drag/dropped into the file input, this adds the drop event to the listener. Also fixed the broken jQuery autofocus (bad selector, wrong trigger).
Fixed the "clear file" button not properly clearing dupesContainer. Added check for which input element triggered the event so that URLs and local files can be handled separately.
For some reason, I have to change version numbers whenever I make a new build, otherwise the injector seems to not recognize that a new build occurred. I forgot to revert this prior to my previous commit.
The APIQuery interface doesn't allow anything but strings so I'm gonna have to ad-hoc this whole thing. It's gonna be ugly, but I hope it'll work.
… and URL handlers
… and URL handlers
atomicBlaze21
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #360
Added check to determine if event is triggered by the file input or the URL textbox.
Added POST fetch to properly query the IQDB endpoint with the image file. (The APIQueryEntry type only accepts strings, numbers, and boolean. It can't handle form data or blobs, so I couldn't use the E621 API module.)
Fixed "Clear File" button not properly clearing the duplicate container.