#6595 closed enhancement (fixed)
Add a new data store for Google search
Reported by: | sos | Owned by: | Jared Jurkiewicz |
---|---|---|---|
Priority: | high | Milestone: | 1.2 |
Component: | DojoX Data | Version: | 1.1.0 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Google make their search API available as a RESTful service. A dojox.data store should be written to access it.
Attachments (9)
Change History (19)
Changed 13 years ago by
Attachment: | GoogleSearch.patch added |
---|
comment:1 Changed 13 years ago by
I've discovered a bug with the store, relating to using non-web based search, e.g. blogs. This will be fixed tonight, and a new patch submitted
Changed 13 years ago by
Attachment: | GoogleSearch_multipleStores.patch added |
---|
Update patch that implements multiple different stores, one for each type of search
comment:2 Changed 13 years ago by
Will look at this soon! This sounds really cool. Thanks for doing the initial work on it. :)
comment:3 Changed 13 years ago by
Looked at it a bit, fixed a few things, added a demo that attaches it to grid, tried to remove the usage of simpleFetch, though I don't know if that is necessary
Overall, I think you have some bugs in the paging mechanism. Start/count don't work quite as expected when it is trying to get subsets. This shows up particularly in grid. For some reason it only shows the first page. Could you take a look at your paging logic again and see why it is acting funny? I looked over it some, and will do so more later, but I haven't figured it quite out yet.
Changed 13 years ago by
Attachment: | dojox.data.GoogleSearchStore_20080501.patch added |
---|
My version of the patch with extra demo + some attempted fixes.
comment:4 Changed 13 years ago by
Sure, will look in to it. Do you know of any specific tests, or combination of tests, that it fails? I included all the standard tests, which it passes. Perhaps we need to extend the basic set of tests for more coverage.
Changed 13 years ago by
Attachment: | dojox.data.GoogleSearchStore_20080501_2.patch added |
---|
Updated patch with paging errors fixed
Changed 13 years ago by
Attachment: | dojox.data.GoogleSearchStore_20080501_3.patch added |
---|
Patch with more fixes, and tests for Image gallery and Slideshow
comment:5 Changed 13 years ago by
Patch dojox.data.GoogleSearchStore_20080501_3.patch also includes a fix for dojox.image.Gallery, where it was broken if a store other than one of the Flickr stores was used.
Changed 13 years ago by
Attachment: | dojox.data.GoogleSearchStore_20080501_4.patch added |
---|
More cleanup. Fixed a few accidental globals, one invalid test, some structure, and jslint complaints.
comment:6 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Changed 13 years ago by
Attachment: | GoogleSearch_args.patch added |
---|
Fixes checking of contructor args
Changed 13 years ago by
Attachment: | GoogleFeedStore_Initial.patch added |
---|
Patch that implements a store that accesses Google's Feed API. Use this instead of the previous patch!
comment:8 Changed 13 years ago by
(In [13592]) Minor updates to GoogleSearchStore? + the GoogleFeedStore? extension. Many thanks ot Shane for his hard work on them. refs #6595 \!strict
comment:9 Changed 13 years ago by
Added in Shane's addition after a quick review. Tested it all on FF 2, IE 6, Seamonkey 1.1.2, and Safari 3.1
comment:10 Changed 13 years ago by
Milestone: | → 1.2 |
---|
Changed 13 years ago by
Attachment: | GoogleSearch_dijitRequire.patch added |
---|
Performance improvement for the demos
Patch to implement this feature