Project

General

Profile

Bug #2062

Fixed JSON in mod_uploadprogress

Added by double about 10 years ago. Updated about 3 years ago.

Status:
Obsolete
Priority:
Normal
Assignee:
-
Category:
mod_uploadprogress
Target version:
Start date:
2009-09-03
Due date:
% Done:

0%

Estimated time:
Missing in 1.5.x:
Yes

Description

JSON does not contain "new Object({...}) but only {...}
http://en.wikipedia.org/wiki/JSON

Otherwise mod_uploadprogress does not work with JS libraries
like Mootools, jQuery, ...

Easy patch:

473c473
< buffer_append_string_len(b, CONST_STR_LEN("new Object({ 'state' : 'starting' })\r\n"));
---

buffer_append_string_len(b, CONST_STR_LEN("{ 'state' : 'starting' }\r\n"));

481c481
< buffer_copy_string_len(b, CONST_STR_LEN("new Object({ 'state' : "));
---

buffer_copy_string_len(b, CONST_STR_LEN("{ 'state' : "));

498c498
< buffer_append_string_len(b, CONST_STR_LEN("})\r\n"));
---

buffer_append_string_len(b, CONST_STR_LEN("}\r\n"));

History

#1

Updated by gstrauss over 3 years ago

  • Target version set to 1.5.0
#2

Updated by gstrauss about 3 years ago

  • Missing in 1.5.x set to Yes
#3

Updated by gstrauss about 3 years ago

  • Status changed from New to Obsolete

Also available in: Atom