Ticket #1647: videolist.patch

mythvideo/mythvideo/videolist.h

@@ -45 +45 @@
         GenericTree *addDirNode(GenericTree *where_to_add,
                                                         const QString& dname);
         GenericTree *addFileNode(GenericTree *where_to_add,
-                                                const QString& fname, int id);
+                                 const QString& fname, const QString& title, int id);
 
         bool m_ListUnknown;
         bool m_LoadMetaData;

mythvideo/mythvideo/videolist.cpp

@@ -221 +221 @@
         file_string.remove(0, prefix.length());
         if (flatlist) 
         {
-            video_tree_root->addNode(file_string, count, true);
+            video_tree_root->addNode(myData->Title(), count, true);
             nitems++;
         }
         else 
         {
-            where_to_add = addFileNode(where_to_add, file_string, count);
+            where_to_add = addFileNode(where_to_add, file_string, myData->Title(), count);
         }
         metas.append(*myData);
         delete myData;
@@ -336 +336 @@
         }
 
         Metadata *myData = new Metadata;
-        // See if we can find this filename in DB
+        // See if we can/should find this filename in DB
         myData->setFilename(file_string);
-        if (m_LoadMetaData) {
-            if(!myData->fillDataFromFilename()) {
-                // No, fake it.
-                QString base_name = file_string.section("/", -1);
-                myData->setTitle(base_name.section(".", 0, -2));
-            }
-        }
-        else
-        {
+        if (!m_LoadMetaData || !myData->fillDataFromFilename()) {
+            // No, fake it.
             QString base_name = file_string.section("/", -1);
             myData->setTitle(base_name.section(".", 0, -2));
-
         }
 
         metas.append(*myData); 
@@ -358 +350 @@
         {
             file_string.remove(0, prefix.length());
             if (!flatlist) {
-                where_to_add = addFileNode(where_to_add, file_string, i);
+                where_to_add = addFileNode(where_to_add, file_string, myData->Title(), i);
             }
             else {
-                video_tree_root->addNode(file_string, i, true);
+                video_tree_root->addNode(myData->Title(), i, true);
                 nitems++;
             }
         }
@@ -423 +415 @@
 }
 
 GenericTree *VideoList::addFileNode(GenericTree *where_to_add,
-                                                const QString& fname, int index)
+                                    const QString& fname, const QString& title, int index)
 {
     int a_counter = 0;
     GenericTree *sub_node;
@@ -434 +426 @@
     {
         if(a_counter + 1 >= (int) list.count()) // video
         {
-            QString title = (*an_it);
-            sub_node = where_to_add->addNode(title.section(".",0,-2),
+            sub_node = where_to_add->addNode(title,
                     index, true);
             sub_node->setAttribute(0, ORDER_ITEM);
             sub_node->setOrderingIndex(0);