Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I've developed a module called Linked Data Import that can be used to import remote Linked Data content (using either content negotiation or SPARQL) and create nodes from that content. The module depends heavily on the Feeds module and is primarily a set of plugins for Feeds.
The README and INSTALL files with the module contain further information.
Sandbox project: http://drupal.org/sandbox/milesw/1085078
Comment | File | Size | Author |
---|---|---|---|
#1 | milesworthington-rdfimporter-v0.1-0-g6ed343b.tar_.gz | 17.79 KB | milesw |
Comments
Comment #1
milesw CreditAttribution: milesw commentedAdding tarball with latest module code from GitHub.
Comment #2
apadernoHello, and thank you for applying for a CVS account.
As per requirements, the motivation needs to include a description of the module features, and a comparison with the existing solutions; the module feature description should be longer than two sentences.
Comment #3
apadernoI set the wrong status.
Comment #4
milesw CreditAttribution: milesw commentedSorry, I thought it would be easier just to point to GitHub and the README where those questions are answered. I'll paste it here...
Comment #5
zzolo CreditAttribution: zzolo commentedHi. Please read all the following and the links provided as this is very important information about your CVS Application:
Drupal.org has moved from CVS to Git! This is a very significant change for the Drupal community and for your application. Please read the following documentation on how this affects and benefits you and the application process:
Migrating from CVS Applications to (Git) Full Project Applications
Comment #6
milesw CreditAttribution: milesw commentedThe module is now located in this Git sandbox:
http://drupal.org/sandbox/milesw/1085078
Comment #7
jthorson CreditAttribution: jthorson commentedPlease note step 5 at the Migrating CVS Applications Link.
Comment #8
sreynen CreditAttribution: sreynen commentedI'm updating the title on this to match the project, hopefully making it a little easier for interested reviewers to find.
Comment #9
Anonymous (not verified) CreditAttribution: Anonymous commentedWow, this has lingered in the queue for a long time.
I am reviewing the 7.x version of the module as found on GitHub.
This module is pretty solid in a lot of ways:
Now to the work...
Style issues:
* Gather sample data for all an individual's properties and save it in the batch.
is 85 characters. In Komodo IDE (or Komodo Edit), there is a faint line at 80 chars, maybe there is in your editor too.Some potential security issues:
RdfImporterParser.inc
RdfImporterNodeProcessor.inc
need a node_access tag? I'm not familiar enough with the feeds_node_item table and the data it stores.Looking forward to reviewing again, we'll make sure it doesn't take a year this time ;)
Comment #10
sreynen CreditAttribution: sreynen commentedlinclark, it's great to have a topic expert doing reviews, thanks! If you do another round after these updates, please look at the sandbox project rather than the Github version, since that's what will eventually become the full project. I'm guessing they're the same right now, but maybe not.
milesw, make sure you're doing your updates in the sandbox project. If you want to keep Github updated, that's great too.
I just added the sandbox link to the issue summary to make it a little easier to find.
Comment #11
milesw CreditAttribution: milesw commentedThanks, linclark, for the great code review. And thanks jthorson and sreynen for nudging this along.
Based on a suggestion from linclark, I've renamed the project from "rdfimporter" to "ld_import" (Linked Data Import) to better represent the features of the module.
I've committed a number of changes to the sandbox repository based on the comments in #9...
Style issues:
Security issues:
Hope I've covered all the issues. I did check the ARC2 path in the latest RDFx module and it appears to be the same as what I'm using.
Comment #11.0
milesw CreditAttribution: milesw commentedAdded link to sandbox.
Comment #12
klausiComment #13
MiSc CreditAttribution: MiSc commented@milesw has been contacted to ask if the application is abandoned.
http://drupal.org/node/894256
Comment #14
milesw CreditAttribution: milesw commentedYup, closing the application. Not really abandoned, but probably not enough interest for Drupal.org, so no need to waste any more time reviewing. Thanks to those who already have.
Comment #14.0
milesw CreditAttribution: milesw commentedChanged module name, removed GitHub link to avoid confusion.
Comment #15
apaderno