RDF UI depends on these modules, so need to add to info file.

CommentFileSizeAuthor
#4 1016420-4-rdfx-dependency.patch347 byteslinclark
#2 1016420-2-rdfx-dependency.patch356 byteslinclark

Comments

Anonymous’s picture

Anonymous (not verified) commented
Title: RDF UI depends on Evoc and RDFx » RDF UI depends on RDFx

Actually, it only depends on RDFx. A one line fix, I'll be showing someone how to patch today, so I'll use this as an example.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs review
StatusFileSize
new 1016420-2-rdfx-dependency.patch356 bytes

This is a simple patch that adds the dependency.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Needs work

Nice try, but needs a little more work.

+++ b/modules/rdfui/rdfui.infoundefined
@@ -3,4 +3,5 @@ description = User interface for altering the RDF mapping of Drupal data structu
+dependencies[] =  rdfx, foobar

This has extra whitespace and doesn't follow the proper syntax for dependencies. Each should be listed on their own line.

Powered by Dreditor.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Needs review
StatusFileSize
new 1016420-4-rdfx-dependency.patch347 bytes

Great, thanks for the review. Can't believe I missed that.

Here is a new patch.

cygri’s picture

cygri commented
Status: Needs review » Reviewed & tested by the community

Looks ok.

This patch adds a dependency on rdfx to rdfui.

Anonymous’s picture

Anonymous (not verified) commented
Status: Reviewed & tested by the community » Fixed

Fixed in commit http://drupalcode.org/project/rdf.git/commit/afd87e8

Thanks to cygri for the review!

Anonymous’s picture

Anonymous (not verified) commented

#801690: Integrate the Libraries API

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

milesw’s picture

milesw commented
Project: Resource Description Framework (RDF) » RDF Extensions