Skip to Content.
Sympa Menu

cc-devel - Re: [cc-devel] Bugs on PdfLicenseManager

cc-devel AT lists.ibiblio.org

Subject: Developer discussion for Creative Commons technology and tools

List archive

Chronological Thread  
  • From: Nathan Yergler <nathan AT creativecommons.org>
  • To: Juan José Marín Martínez <juanj.marin AT juntadeandalucia.es>
  • Cc: cc-devel AT lists.ibiblio.org
  • Subject: Re: [cc-devel] Bugs on PdfLicenseManager
  • Date: Tue, 9 Jun 2009 08:46:23 -0700

Hi Juan,

Thanks for the patch. That does seem like a bug and the inclusion of
the web statement is a good feature request. I've logged both of
those in our issue tracker (http://code.creativecommons.org/issues).

Thanks!

Nathan

2009/6/6 Juan José Marín Martínez <juanj.marin AT juntadeandalucia.es>:
> Hi,
>
> For testing my patch for adding support for evince, the GNOME document
> viewer,
> I try the PdfLicenseManager.
>
> I've realised that this program doesn't tag the pdfs correctly. Possibly the
> example PDF on [1], that I mentioned on a previous message on this list [2]
> was tagged with this program. I was told that the licence uri must be a a
> resource, not a literal.
>
> I've written a quick and dirty patch for this [3]
>
> Others bugs/ missing features I found are:
>
> 1) It doesn't let you to enter the web statement of your choice. It sets the
> web statement with the license uri. This is a bug, FMHO and it asks loudly
> for
> a feature ;). The web statement must be optional and be choosen by the user
> (additionally the uri can be checked syntactically and even checked for
> availability)
>
> 2) Only tag the with lang='x-default'. It would be nice to tag other
> languages
> too.
>
> 3) It must let you to choose the Jurisdiction of your license
>
> 4) The interface is only in English.
>
>   -- Juanjo Marín
>
>
> [1] http://bugzilla.gnome.org/show_bug.cgi?id=349173#c6
> [2] https://lists.ibiblio.org/sympa/arc/cc-devel/2009-June/001385.html
> [3]
> Index: pdflicense/XmpManager.java
> ===================================================================
> --- pdflicense/XmpManager.java  (revisión: 12728)
> +++ pdflicense/XmpManager.java  (copia de trabajo)
> @@ -87,11 +87,11 @@
>
>                nl = doc.getElementsByTagName("cc:license");
>                if (nl!=null) {
> -                       Node cclicense = nl.item(0);
> +                       Element cclicense = (Element)nl.item(0);
>                        if (cclicense != null) {
> -                               Node text = cclicense.getFirstChild();
> +                               Attr text =
> cclicense.getAttributeNode("rdf:resource");
>                                if (text != null) {
> -                                       lic.setCCLicense(
> text.getNodeValue() );
> +                                       lic.setCCLicense( text.getValue() );
>                                }
>                        }
>                }
> @@ -175,7 +175,7 @@
>                        Element cclicense = doc.createElement("cc:license");
>                        descr.setAttribute("rdf:about", "");
>                        descr.setAttribute("xmlns:cc",
> "http://creativecommons.org/ns#";);
> -                      
> cclicense.appendChild(doc.createTextNode(lic.getUrl()) );
> +                       cclicense.setAttribute("rdf:resource", lic.getUrl()
> );
>                        //cclicense.setAttribute("rdf:", lic.getUrl() );
>                        descr.appendChild(cclicense);
>                        rdf.appendChild(descr);
> @@ -313,7 +313,7 @@
>                s.append(" </rdf:Description>\n");
>
>                s.append(" <rdf:Description rdf:about=''
> xmlns:cc='http://creativecommons.org/ns#'>\n");
> -               s.append("  <cc:license>"+lic.getUrl()+"</cc:license>\n");
> +               s.append("  <cc:license rdf:resource="+lic.getUrl()+"/>\n");
>                s.append(" </rdf:Description>\n");
>
>                s.append("</rdf:RDF>\n");
>
>
>




Archive powered by MHonArc 2.6.24.

Top of Page