Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Failure in posixify when no data #24

Closed
kent37 opened this issue Feb 21, 2015 · 1 comment
Closed

Failure in posixify when no data #24

kent37 opened this issue Feb 21, 2015 · 1 comment
Labels

Comments

@kent37
Copy link
Contributor

kent37 commented Feb 21, 2015

If a query with a date column returns no data, posixify() fails with

Error in if (any(regexpr("^[[:digit:]]{1,2}/[[:digit:]]{1,2}/[[:digit:]]{4}$",  : 
  missing value where TRUE/FALSE needed

For example
read.socrata('https://data.cityofboston.gov/resource/awu8-dc52.csv?property_id=177808')

@tomschenkjr
Copy link
Contributor

Thank you. Also realizing that we never had a unit test that covered that issue. We'll look at #25 and integrate into our builds. I suspect our next stable CRAN submission will be in March.

tomschenkjr pushed a commit that referenced this issue Feb 23, 2015
Pull request which closes #17, resolves #18, fixes #22, close #24, fix #25
dmpe added a commit to dmpe/RSocrata that referenced this issue Jul 19, 2015
dmpe added a commit to dmpe/RSocrata that referenced this issue Jul 23, 2015
dmpe added a commit to dmpe/RSocrata that referenced this issue Dec 24, 2015
Merge pull request Chicago#51 from Chicago/dev

Pull request for v1.6.1
Remove NEWS.md from build to avoid NOTE

Build number bump and date change

R 3.2.1 wants all URLs in canonical form--has been changed

Fix type-o in Coveralls badge
Merge remote-tracking branch 'upstream/dev' into dev

split & improve 4x4 logic (tests pass)
bump to 1.6.3
Add new Floatin Timestamp format for posixify + add test

split validate + test

validate url split test

fix travis

don't run, not do not run
fix travis again

add errorHandling function
and rename getResponse to checkResponse
update test

else if [skip ci]

add not finished test for Chicago#27, Chicago#24
[skip ci]

go over sprint7 branch [skip ci]

text/plain [skip ci]

fix paste -> paste0 because of sep="" which I deleted

delete note.md + update date [skip ci]

geo readme

Merge remote-tracking branch 'upstream/dev' into dev

Merge remote-tracking branch 'origin/dev' into dev

Conflicts:
	DESCRIPTION

add geojson example & vignette (dependencies are suggested, later to be moved to required)

should fix travis

now fix again

second time the same.

give up

give up on travis. wont work

?

fix last time

again

tests and comments (geo too)
update functions, move 4x4 to utils.R
update docu + .md files

add geojson support. not finished

Merge pull request Chicago#53 from dmpe/dev

Split functions and other smaller improvements
fix leaflet example, not SP object but the list and add a new contributor
fix test [skip ci]

Merge pull request Chicago#56 from dmpe/dev-geojson-big

bind -> rbind.fill; getMetadata; GeoJSON method
Fixed version number scheme

Using building numbers, x.y.z-b. The "z" should only be incremented on bug releases being planned.
Human-readable URLs are not being deprecated

Fixes build url under certain domain cases

Re-added human-readable URL

Added unit tests for broken-out URLs

Removed unnecessary library listings

Cleaned-up documentation

Increment build info

Updated help files

Removed vignettes, updated documentation for style

Example formatting issues, cleaned-up comments

Turned on tests

Examples on read.socrata taking too long, stopped their run

Merge pull request #4 from Chicago/dev

Dev
dmpe added a commit to dmpe/RSocrata that referenced this issue Dec 24, 2015
Merge pull request Chicago#51 from Chicago/dev

Pull request for v1.6.1
Remove NEWS.md from build to avoid NOTE

Build number bump and date change

R 3.2.1 wants all URLs in canonical form--has been changed

Fix type-o in Coveralls badge
Merge remote-tracking branch 'upstream/dev' into dev

split & improve 4x4 logic (tests pass)
bump to 1.6.3
Add new Floatin Timestamp format for posixify + add test

split validate + test

validate url split test

fix travis

don't run, not do not run
fix travis again

add errorHandling function
and rename getResponse to checkResponse
update test

else if [skip ci]

add not finished test for Chicago#27, Chicago#24
[skip ci]

go over sprint7 branch [skip ci]

text/plain [skip ci]

fix paste -> paste0 because of sep="" which I deleted

delete note.md + update date [skip ci]

geo readme

Merge remote-tracking branch 'upstream/dev' into dev

Merge remote-tracking branch 'origin/dev' into dev

Conflicts:
	DESCRIPTION

add geojson example & vignette (dependencies are suggested, later to be moved to required)

should fix travis

now fix again

second time the same.

give up

give up on travis. wont work

?

fix last time

again

tests and comments (geo too)
update functions, move 4x4 to utils.R
update docu + .md files

add geojson support. not finished

Merge pull request Chicago#53 from dmpe/dev

Split functions and other smaller improvements
fix leaflet example, not SP object but the list and add a new contributor
fix test [skip ci]

Merge pull request Chicago#56 from dmpe/dev-geojson-big

bind -> rbind.fill; getMetadata; GeoJSON method
Fixed version number scheme

Using building numbers, x.y.z-b. The "z" should only be incremented on bug releases being planned.
Human-readable URLs are not being deprecated

Fixes build url under certain domain cases

Re-added human-readable URL

Added unit tests for broken-out URLs

Removed unnecessary library listings

Cleaned-up documentation

Increment build info

Updated help files

Removed vignettes, updated documentation for style

Example formatting issues, cleaned-up comments

Turned on tests

Examples on read.socrata taking too long, stopped their run

Merge pull request #4 from Chicago/dev

Dev
dmpe added a commit to dmpe/RSocrata that referenced this issue Dec 24, 2015
Merge pull request Chicago#51 from Chicago/dev

Pull request for v1.6.1
Remove NEWS.md from build to avoid NOTE

Build number bump and date change

R 3.2.1 wants all URLs in canonical form--has been changed

Fix type-o in Coveralls badge
Merge remote-tracking branch 'upstream/dev' into dev

split & improve 4x4 logic (tests pass)
bump to 1.6.3
Add new Floatin Timestamp format for posixify + add test

split validate + test

validate url split test

fix travis

don't run, not do not run
fix travis again

add errorHandling function
and rename getResponse to checkResponse
update test

else if [skip ci]

add not finished test for Chicago#27, Chicago#24
[skip ci]

go over sprint7 branch [skip ci]

text/plain [skip ci]

fix paste -> paste0 because of sep="" which I deleted

delete note.md + update date [skip ci]

geo readme

Merge remote-tracking branch 'upstream/dev' into dev

Merge remote-tracking branch 'origin/dev' into dev

Conflicts:
	DESCRIPTION

add geojson example & vignette (dependencies are suggested, later to be moved to required)

should fix travis

now fix again

second time the same.

give up

give up on travis. wont work

?

fix last time

again

tests and comments (geo too)
update functions, move 4x4 to utils.R
update docu + .md files

add geojson support. not finished

Merge pull request Chicago#53 from dmpe/dev

Split functions and other smaller improvements
fix leaflet example, not SP object but the list and add a new contributor
fix test [skip ci]

Merge pull request Chicago#56 from dmpe/dev-geojson-big

bind -> rbind.fill; getMetadata; GeoJSON method
Fixed version number scheme

Using building numbers, x.y.z-b. The "z" should only be incremented on bug releases being planned.
Human-readable URLs are not being deprecated

Fixes build url under certain domain cases

Re-added human-readable URL

Added unit tests for broken-out URLs

Removed unnecessary library listings

Cleaned-up documentation

Increment build info

Updated help files

Removed vignettes, updated documentation for style

Example formatting issues, cleaned-up comments

Turned on tests

Examples on read.socrata taking too long, stopped their run

Merge pull request #4 from Chicago/dev

Dev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants