fix for dropdown matching by label

craftcms · Jul 5, 2016 · 3141168 · 3141168
1 parent 5113318
commit 3141168
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/feedme/services/FeedMeService.php b/feedme/services/FeedMeService.php
@@ -245,10 +245,12 @@ public function deleteLeftoverEntries($settings, $feed, $processedEntries, $resu
             $entriesToDelete = $criteria->find();
 
             try {
-                if (!craft()->entries->deleteEntry($entriesToDelete)) {
-                    FeedMePlugin::log('FeedMeError - Something went wrong while deleting entries.', LogLevel::Error, true);
-                } else {
-                    FeedMePlugin::log($feed->name . ': The following entries have been deleted: ' . print_r($deleteIds, true) . '.', LogLevel::Error, true);
+                if ($entriesToDelete) {
+                    if (!craft()->entries->deleteEntry($entriesToDelete)) {
+                        FeedMePlugin::log('FeedMeError - Something went wrong while deleting entries.', LogLevel::Error, true);
+                    } else {
+                        FeedMePlugin::log($feed->name . ': The following entries have been deleted: ' . print_r($deleteIds, true) . '.', LogLevel::Error, true);
+                    }
                 }
             } catch (\Exception $e) {
                 FeedMePlugin::log($feed->name . ': FeedMeError: ' . $e->getMessage() . '.', LogLevel::Error, true);

diff --git a/feedme/services/FeedMe_FieldsService.php b/feedme/services/FeedMe_FieldsService.php
@@ -193,7 +193,7 @@ public function prepDropdown($data, $field) {
 
         // find matching option label
         foreach ($options as $option) {
-            if ($data == $option['label']) {
+            if ($data == $option['value']) {
                 $fieldData = $option['value'];
                 break;
             }