We have been migrating some media from a legacy system which has offered up some interesting methods of storying ISSUU media. We needed to properly parse these so have added some additional patterns to the parser.

Patch to follow.

CommentFileSizeAuthor
#1 issuu-parse-additional-patterns-2447459-1.patch2.59 KBmsmithcti
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

msmithcti’s picture

msmithcti CreditAttribution: msmithcti commented
Status: Active » Needs review
FileSize
issuu-parse-additional-patterns-2447459-1.patch2.59 KB

This patch adds support for the following:

  • http://issuu.com/<user>/docs/<doctitle> style URLs as currently, but with support for query strings (doesn't include the query string in the document name)
  • SWF embedded links, e.g. http://static.issuu.com/webembed/viewers/style1/v1/IssuuViewer.swf?username=<user>&docName=<doctitle>
  • JS embedded divs that use a numeric representation of a user/doctitle. E.g. <div data-configid="2345/6789".... This required a call to the ISSUU server in order to convert the numeric IDs.
mallezie’s picture

mallezie
Primary language Dutch
Location Loenhout
CreditAttribution: mallezie commented

Thanks! At first sight this seems a nice addition, i'll test this some further first, and then i'll commit your patch.

mallezie’s picture

mallezie
Primary language Dutch
Location Loenhout
CreditAttribution: mallezie commented
Status: Needs review » Fixed

Thanks, i tested the patterns you added, and checked the code, everything seems great. so i committed this, by tomorrow this will be in the new dev release.

Status: Fixed » Closed (fixed)

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