search

jbach / jbx_multiple_image_upload

Multiple Image Upload Plugin for Textpattern
https://packagist.org/packages/jbach/jbx_multiple_image_upload
GNU General Public License v2.0
1 stars 3 forks source link

Error on upload, showing login for TXP 4.5.4 #1

Closed wavesource closed 10 years ago

wavesource commented 10 years ago

_ss_4

jpdupont commented 10 years ago

Hello, same issue on last nightly 4.6 ... These login screen is shown for each image ... None image is uploaded.

jbach commented 10 years ago

Interestingly, the version of Flash that is bundled with Chrome does submit Cookies, which is why this plugin works perfectly fine in Chrome. If I turn off the internal version of Flash, therefore falling back to the system one, I get the same error, as Textpattern does not see the request made by Uploadify as authenticated.

Try using Chrome, you will see the plugin working. (Not actually a solution)

Two options:

  1. Handle the upload with an intermediary script, which returns the $_FILES['Filedata'], then trigger the image insertion in Texpattern with an additional AJAX call
  2. Drop Uploadify. Or at least, the flash variant. This would mean dropping support for IE < 10

I’m in favor of option 2. I’m going to try to write up a quickfix with option 1 first, though.

EDIT: I see abl_droploader seems to cover option 2. So maybe this plugin should proceed with Flash and could be used if abl_droploader does not fit the scenario.

jpdupont commented 10 years ago

I'm in favor to drop the flash part of the plugin, to some HTML5 Ajax / jquery solution like this :

https://github.com/blueimp/jQuery-File-Upload

Le 13/06/2014 15:07, Jonas Bach a écrit :

Interestingly, the version of Flash that is bundled with Chrome does submit Cookies, which is why this plugin works perfectly fine in Chrome. If I turn off the internal version of Flash, therefore falling back to the system one, I get the same error, as Textpattern does not see the request made by Uploadify as authenticated.

Try using Chrome, you will see the plugin working. (Not actually a solution)

Two options:

  1. Handle the upload with an intermediary script, which returns the |$_FILES['Filedata']|, then trigger the image insertion in Texpattern with an additional AJAX call
  2. Drop Uploadify. Or at least, the flash variant. This would mean dropping support for IE < 10

I’m in favor of option 2. I’m going to try to write up a quickfix with option 1 first, though.

— Reply to this email directly or view it on GitHub https://github.com/jbach/jbx_multiple_image_upload/issues/1#issuecomment-46008608.

Jean-Pol Dupont

jbach commented 10 years ago

Should be fixed with https://github.com/jbach/jbx_multiple_image_upload/commit/2ebf00e1df54150b723649fdd8f8f71ecfc64b59