diff -r 735c7ea1d2a9 -r 4a84d7e444d8 vymmodel.cpp
--- a/vymmodel.cpp	Tue Jan 05 11:23:12 2010 +0000
+++ b/vymmodel.cpp	Thu Jan 21 11:56:57 2010 +0000
@@ -14,6 +14,7 @@
 #include "mainwindow.h"
 #include "misc.h"
 #include "parser.h"
+#include "process.h"
 
 #include "warningdialog.h"
 #include "xlinkitem.h"
@@ -1550,14 +1551,21 @@
 		return QString();
 }
 
-void VymModel::setNote(const QString &s)	//FIXME-2 savestate missing	// FIXME-2 call to VM::updateNoteFlag missing (fix signal handling here)
-{
+void VymModel::setNote(const QString &s)
+{
+	cout << "VM::setNote\n";
 	TreeItem *selti=getSelectedItem();
 	if (selti) 
 	{
+		emitNoteHasChanged(selti);
+		saveState(
+			selti,
+			"setNote (\""+selti->getNote()+"\")", 
+			selti,
+			"setNote (\""+s+"\")", 
+			QString("Set note of %1 ").arg(getObjectName(selti)) );
+	}
 		selti->setNote(s);
-		emitNoteHasChanged(selti);
-	}
 }
 
 QString VymModel::getNote()
@@ -1569,6 +1577,42 @@
 		return QString();
 }
 
+void VymModel::findDuplicateURLs()	// FIXME-3 needs GUI
+{
+	// Generate map containing _all_ URLs and branches
+	QString u;
+	QMap <QString,BranchItem*> map;
+	BranchItem *cur=NULL;
+	BranchItem *prev=NULL;
+	nextBranch(cur,prev);
+	while (cur) 
+	{
+		u=cur->getURL();
+		if (!u.isEmpty() )
+			map.insertMulti (u,cur);
+		nextBranch(cur,prev);
+	}
+
+	// Extract duplicate URLs
+	QMap <QString, BranchItem*>::const_iterator i=map.constBegin();
+	QMap <QString, BranchItem*>::const_iterator firstdup=map.constEnd();	//invalid
+	while (i != map.constEnd())
+	{
+		if (i!=map.constBegin() && i.key()==firstdup.key())
+		{
+			if (  i-1==firstdup )
+			{
+				cout << firstdup.key().toStdString() << endl;
+				cout << " - "<< firstdup.value() <<" - "<<firstdup.value()->getHeadingStd()<<endl;
+			}	
+			cout << " - "<< i.value() <<" - "<<i.value()->getHeadingStd()<<endl;
+		} else
+			firstdup=i;
+
+		++i;
+	}
+}
+
 BranchItem* VymModel::findText (QString s, bool cs)   
 {
 	if (!s.isEmpty() && s!=findString)
@@ -1679,7 +1723,7 @@
 		return QString();
 }
 
-QStringList VymModel::getURLs()	
+QStringList VymModel::getURLs(bool ignoreScrolled)	
 {
 	QStringList urls;
 	BranchItem *selbi=getSelectedBranch();
@@ -1687,7 +1731,8 @@
 	BranchItem *prev=NULL;
 	while (cur) 
 	{
-		if (!cur->getURL().isEmpty()) urls.append( cur->getURL());
+		if (!cur->getURL().isEmpty()  && !(ignoreScrolled && cur->hasScrolledParent(cur) )) 
+			urls.append( cur->getURL());
 		cur=nextBranch (cur,prev,true,selbi);
 	}	
 	return urls;
@@ -2888,7 +2933,77 @@
 		{
 			QRegExp rx("(\\d+)");
 			if (rx.indexIn(url) !=-1)
-				cout << "VM::getBugzillaData bug="<<rx.cap(1).toStdString()<<endl;
+			{
+				QString bugID=rx.cap(1);
+				cout << "VM::getBugzillaData bug="<<bugID.toStdString()<<endl;
+
+				QString script="test/vym-bug.pl";
+				/*
+				QProgressDialog progress2("Copying files...", "Abort Copy", 0, 40, mainWindow);
+				progress2.setWindowModality(Qt::WindowModal);
+				progress2.setValue (3);
+				progress2.update();
+				QProgressDialog progress ("Contacting Bugzilla...","empty",0,0);
+				progress.setCancelButton (NULL);
+				progress.setWindowModality(Qt::WindowModal);
+				progress.show();
+				progress.update();
+*/
+				Process p;
+				p.start (script,QStringList()<<bugID);
+				if (!p.waitForStarted())
+				{
+					cout <<"VM::getBugzillaData couldn't start "<<script.toStdString()<<endl;
+					return;
+				}	
+				if (!p.waitForFinished())
+				{
+					cout <<"VM::getBugzillaData couldn't finish "<<script.toStdString()<<endl;
+					//progress.hide();
+					return;
+				}
+				//progress.hide();
+				//QByteArray result=p.readAll();
+				QString result=p.getStdout();
+				while (result.endsWith("\n")) result.chop(1); 
+				//cout << QString(result).toStdString()<<endl;
+				QString err=p.getErrout();
+				if (!err.isEmpty())
+				{
+					cout << "VM::getBugzillaData Error:\n";
+					cout <<err.toStdString()<<endl;
+				}
+				else if (!result.isEmpty())
+				{
+					QString heading,cdate,mdate,state,whiteboard;
+					QRegExp re("short_desc:(.*)\n");
+					re.setMinimal(true);
+					if (re.indexIn (result) !=-1) heading=re.cap(1);
+
+					re.setPattern ("creation_ts:(.*)\\s");
+					if (re.indexIn (result) !=-1) cdate=re.cap(1);
+
+					re.setPattern ("delta_ts:(.*)\\s");
+					if (re.indexIn (result) !=-1) mdate=re.cap(1);
+
+					re.setPattern ("bug_status:(.*)\n");
+					if (re.indexIn (result) !=-1) state=re.cap(1);
+
+					re.setPattern ("status_whiteboard:(.*)\n");
+					if (re.indexIn (result) !=-1) whiteboard=re.cap(1);
+
+					setHeading (bugID + " - " + heading);
+					cout << "VM: heading="<<heading.toStdString()<<endl;
+					cout << "VM:   cdate="<<cdate.toStdString()<<endl;
+					cout << "VM:   mdate="<<mdate.toStdString()<<endl;
+					cout << "VM:   state="<<state.toStdString()<<endl;
+					cout << "VM:  wboard="<<whiteboard.toStdString()<<endl;
+					
+					//cout <<"VM::getBugzillaData  "<<script.toStdString()<<" returned:\n";
+					//cout <<QString(result).toStdString()<<endl;
+				} else	
+					cout << "VM::getBugzillaData "<<script.toStdString()<<"  returned nothing\n";
+			}	
 		}
 	}
 }	
@@ -4342,17 +4457,20 @@
 
 void VymModel::updateNoteFlag()
 {
-	setChanged();
 	TreeItem *selti=getSelectedItem();
 	if (selti)
 	{
+		if (!mapChanged)
+		{
+			setChanged();
+			updateActions();
+		}
+
 		if (textEditor->isEmpty()) 
 			selti->clearNote();
 		else
 			selti->setNote (textEditor->getText());
 		emitDataHasChanged(selti);		
-		emitSelectionChanged();
-
 	}
 }