Capitalize encoding string type and make JSON the default encoding method [#888064]

I was only able to connect using REST with JSON encoding after capitalizing the input and response types in the "_sugarondrupal_rest_json_request" function. Patch attached.

Reference: http://www.ibm.com/developerworks/library/x-sugarcrmrest/index.html#icom...

Comment	File	Size	Author
#4	sugarondrupal-fix-json.patch	5.18 KB	ilo
	json.patch	1.51 KB	ngstigator

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

ilo CreditAttribution: ilo commented 16 December 2010 at 23:21

Status:

Needs review

» Needs work

Thanks chris_five, I'll take a look, but current non-capitalized strings are working for me, so I guess it requires a more in-depth research on SugarCRM versions. What is yours?

Comment #2

ilo CreditAttribution: ilo commented 19 December 2010 at 20:53

Actually, after some tests in a SugarCRM Enterprise 6.1 setup I can confirm this issue (however it is working non-capitalized in a 6.0.3-beta). I've verified that being capitalized also works on 6.0 and 5.5, so I'll address this issue in the next commit.

Comment #3

ilo CreditAttribution: ilo commented 20 December 2010 at 23:39

Title:	Cannot connect with REST/JSON	» Capitalize encoding string type and make JSON the default encoding method
Status:	Needs work	» Fixed

I've changed JSON to be the default encoding, and also capitalized it after verifying that it works capitalized where I was using non-capitalized strings. Thanks for the tip, chris_five, I've committed as http://drupal.org/cvs?commit=467440.