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

Bad function in example/book.js #88

Open
librarywebchic opened this issue Dec 2, 2015 · 6 comments
Open

Bad function in example/book.js #88

librarywebchic opened this issue Dec 2, 2015 · 6 comments

Comments

@librarywebchic
Copy link

I'm trying to work from https://github.com/linkeddata/rdflib.js/blob/master/example/people/book.js

But there is a function
kb.load(uri) which doesn't work.

I get an error that kb.load isn't a function

Has this function been deprecated or renamed?

@melvincarvalho
Copy link
Member

@bblfish did you originally write this code? If so, would you mind taking a look?

@bblfish
Copy link
Contributor

bblfish commented Dec 3, 2015

It's too far in the past for me to be any better to work this out. I imagine over the past 3 years things have changed a lot..

@librarywebchic
Copy link
Author

I was hoping since there isn't documentation on what all the classes and functions are someone might know if this function has been deprecated or renamed.

@melvincarvalho
Copy link
Member

I've never used this function so I dont know what it does. Or what it was supposed to do. Searching for load :

https://github.com/linkeddata/rdflib.js/search?utf8=%E2%9C%93&q=load

This is the only place it's used.

What I normally use is nowOrWhenFetched to get a document.

e.g. https://github.com/linkeddata/rdflib.js/blob/master/example/people/book.js#L54

@bblfish perhaps you would want to consider removing this example or changing it, if it no longer works?

@bblfish
Copy link
Contributor

bblfish commented Dec 6, 2015

It's probably meant to be "nowOrWhenFetched"

@librarywebchic
Copy link
Author

So in other linked data libraries, a load function adds the URI you pass to it the existing graph. Expanding the scope of the graph.

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

No branches or pull requests

3 participants