JSON output bug fixes and improvements
icylace - October 28, 2009 - 04:40
| Project: | Views Datasource |
| Version: | 6.x-1.0-alpha3 |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | allisterbeharry |
| Status: | closed |
Jump to:
Description
This is an update to my previous patch (#605980) that has the following improvements for JSON output:
1. An improved method for placing commas between fields that takes into account empty fields that get skipped over.
2. Views UI previewing wraps the JSON preview within a tag and also formats HTML values to be displayed as regular text.
3. A much-improved views_json_get_json() function that is leaner and hopefully bug-free.
P.S. What is the right way to make a clean patch ?
| Attachment | Size |
|---|---|
| views_datasource_alpha3_003.patch | 112.47 KB |
#1
Whoops, the referenced issue link is #605980: new functionality for the JSON stuff + more formatting improvements + more code changes (I forgot to type in the brackets) and the "code" tag in #2 wasn't supposed to get interpreted. I should really proof-read my posts :)
#2
Thanks so much for your work and sorry for the delay. Committed to the DRUPAL-6--1 branch. Instructions for creating a patch against CVS are here: http://drupal.org/patch/create. Your patch was ok except for it being relative to a vd0 directory. But graphical IDEs can be instructed to ignore the directories in the patch.
#3
Thank you for making such a wonderful module! I rely on it heavily and I've been continuing my development for it. Soon I'll be submitting a patch that adds support for encoding nested objects and arrays and some other stuff. I'll do my best to make cleaner patches from now on.
#4
Automatically closed -- issue fixed for 2 weeks with no activity.