If the metadata that's stripped out of the files is added into the entity property info array it'll expose these values to projects like Services and RestWS.

patch coming

CommentFileSizeAuthor
#2 getid3-entity-property-info-2782997-2.patch1.38 KBbtopro
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

btopro created an issue. See original summary.

btopro’s picture

btopro CreditAttribution: btopro commented
Status: Active » Needs review
FileSize
getid3-entity-property-info-2782997-2.patch1.38 KB

patch adds support for everything exist tags since I can't figure that out since it's variable and a nested array. Most importantly this brings height, width and duration into scope of projects like RestWS so it's queryable.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented 
  1. +++ b/getid3.module
@@ -298,3 +298,41 @@ function theme_getid3_sample_rate($variables) {
+      'label' => t('Getid3 @id', array('@id' => $property)),

    Maybe just "ID3" since that's the actual metadata spec 

  2. +++ b/getid3.module
@@ -298,3 +298,41 @@ function theme_getid3_sample_rate($variables) {
+ * @see read_time_entity_property_info_alter()

    Should this be "getid3"? 

  3. +++ b/getid3.module
@@ -298,3 +298,41 @@ function theme_getid3_sample_rate($variables) {
+  if (isset($file->metadata[$name])) {
+    return $file->metadata[$name];
+  }

    Shoot, it'd be nice if File entity had a helper for this.