feat: added new functionality to parse first image in content to soci…

…al models' img_url (close #2)
JohnnyTheTank · Jan 3, 2016 · ee3cfeb · ee3cfeb
1 parent 2591668
commit ee3cfeb
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 8 deletions.
diff --git a/bower.json b/bower.json
@@ -4,7 +4,7 @@
   "authors": [
     "Jonathan Hornung <jonathan.hornung@gmail.com>"
   ],
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "rss plugin for apiNG",
   "main": "dist/apiNG-plugin-rss.min.js",
   "moduleType": [],

diff --git a/demo/index.html b/demo/index.html
@@ -19,7 +19,7 @@
             items="5"
             order-by="caption"
             order-reverse="false"
-            aping-rss="[{'path':'http://rss.nytimes.com/services/xml/rss/nyt/Science.xml'}]">
+            aping-rss="[{'path':'http://feeds.feedburner.com/Angular-jsDirectives?utm_source=angular-js.in&utm_medium=website&utm_campaign=content-curation'}]">
     </aping>
     <hr>
     <aping

diff --git a/dist/aping-plugin-rss.min.js b/dist/aping-plugin-rss.min.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "aping-plugin-rss",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "RSS plugin for apiNG",
   "main": "Gruntfile.js",
   "scripts": {

diff --git a/src/aping-rss-directive.js b/src/aping-rss-directive.js
@@ -16,7 +16,7 @@ var jjtApingRss = angular.module("jtt_aping_rss", [])
 
                 var appSettings = apingController.getAppSettings();
 
-                var requests = apingUtilityHelper.parseJsonFromAttributes(attrs.apingRss, apingRssHelper.getThisPlattformString(), appSettings);
+                var requests = apingUtilityHelper.parseJsonFromAttributes(attrs.apingRss, apingRssHelper.getThisPlatformString(), appSettings);
 
                 requests.forEach(function (request) {
 

diff --git a/src/aping-rss-helper.js b/src/aping-rss-helper.js
@@ -7,7 +7,7 @@
  */
 
 jjtApingRss.service('apingRssHelper', ['apingModels', 'apingTimeHelper', 'apingUtilityHelper', function (apingModels, apingTimeHelper, apingUtilityHelper) {
-    this.getThisPlattformString = function () {
+    this.getThisPlatformString = function () {
         return "rss";
     };
 
@@ -59,7 +59,7 @@ jjtApingRss.service('apingRssHelper', ['apingModels', 'apingTimeHelper', 'apingU
     };
 
     this.getSocialItemByJsonData = function (_item) {
-        var socialObject = apingModels.getNew("social", this.getThisPlattformString());
+        var socialObject = apingModels.getNew("social", this.getThisPlatformString());
 
         //fill _item in socialObject
         $.extend(true, socialObject, {
@@ -76,6 +76,12 @@ jjtApingRss.service('apingRssHelper', ['apingModels', 'apingTimeHelper', 'apingU
             socialObject.text = _item.title || undefined;
         }
 
+        if(_item.content) {
+            var imagesArray = apingUtilityHelper.getFirstImageFromHtml(_item.content);
+            if(imagesArray.length>1) {
+                socialObject.img_url = imagesArray[1];
+            }
+        }
 
         socialObject.date_time = _item.publishedDate ? new Date(_item.publishedDate) : undefined;
         socialObject.timestamp = socialObject.date_time ? socialObject.date_time.getTime() : undefined;