Item13188: WysiwygPlugin upload rest handler is missing - Upload fails with 404
Priority: Urgent
Current State: Closed
Released In: 2.0.0
Target Release: major
Attachment upload from Wysiwyg /
TinyMCE editor fails with 404.
The plugin does not register an upload handler:
ERROR: (404) Invalid REST invocation - /WysiwygPlugin/upload does not refer to a known handler
- Edit an existing topic
- Click on paperclip, choose upload attachment
- Select a file, press upload
- Fails with 404
Confirmed locally and on trunk.foswiki.org
--
GeorgeClark - 06 Jan 2015
Crawford. Your checkin deleted the upload handler without providing an alternative:
commit bba8feccb71175a0e4907aba001b7be9cc13fbfb
Author: crawford <git@c-dot.co.uk>
Date: Fri Nov 7 14:44:47 2014 +0000
Item12705: open a can of worms, and worms will escape. Spacing in verbatim fixed, and a lot of confusion about character sets (not mine, thankfully) refactored out. There's still an issue with the TMLhtml class causing random roundtrip tests to fail, but i think it's a problem with the tests, foremost. There is one expected-to-fail test that covers it.
--
GeorgeClark - 06 Jan 2015
It's a deeper issue than at first appears.
Done, though upload doesn;t seem to save any data. Need to look again.
--
CrawfordCurrie - 07 Jan 2015
I've "cleaned" and have tried to get wysiwyg working here, and notice several problems.
- It's back to using the
bin/rest/WysiwygPlugin/upload
It lost the change I made to use bin/upload
- It does a GET to run the upload, rather than a POST
- Also check_manifest found a bunch of files in git that are not in the MANIFEST file.
--
GeorgeClark - 08 Jan 2015
- All can say is "no it isn't"
- Erm. No, it doesn't! Try clearing the browser cache.
- We don't ship .uncompressed.js, do we?
--
CrawfordCurrie - 08 Jan 2015