Skip to content

Commit 5e8feb6

Browse files
committed
set full path as name on browser favourites created by provider
1 parent 6c1a771 commit 5e8feb6

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

src/core/qgsdataitem.cpp

+1
Original file line numberDiff line numberDiff line change
@@ -1196,6 +1196,7 @@ QVector<QgsDataItem*> QgsFavouritesItem::createChildren( const QString& favDir )
11961196
QgsDataItem * item = provider->createDataItem( favDir, this );
11971197
if ( item )
11981198
{
1199+
item->setName( favDir );
11991200
children.append( item );
12001201
}
12011202
}

src/providers/grass/qgsgrassprovidermodule.cpp

+7-7
Original file line numberDiff line numberDiff line change
@@ -409,7 +409,7 @@ QIcon QgsGrassMapsetItem::icon()
409409
}
410410
else if ( mGrassObject.locationIdentical( QgsGrass::getDefaultLocationObject() ) )
411411
{
412-
if ( QgsGrass::instance()->isMapsetInSearchPath( mName ) )
412+
if ( QgsGrass::instance()->isMapsetInSearchPath( mGrassObject.mapset() ) )
413413
{
414414
return QgsApplication::getThemeIcon( "/grass_mapset_search.png" );
415415
}
@@ -509,7 +509,7 @@ QVector<QgsDataItem*> QgsGrassMapsetItem::createChildren()
509509
// if topo version does not match GRASS lib version
510510
int topoMajor = 0;
511511
int topoMinor = 0;
512-
bool gotTopoVersion = QgsGrass::topoVersion( mGrassObject.gisdbase(), mGrassObject.location(), mName, name, topoMajor, topoMinor );
512+
bool gotTopoVersion = QgsGrass::topoVersion( mGrassObject.gisdbase(), mGrassObject.location(), mGrassObject.mapset(), name, topoMajor, topoMinor );
513513
QgsDebugMsg( QString( "name = %1 topoMajor = %2 topoMinor = %3" ).arg( name ).arg( topoMajor ).arg( topoMinor ) );
514514
QString topoError;
515515
if ( !gotTopoVersion )
@@ -540,7 +540,7 @@ QVector<QgsDataItem*> QgsGrassMapsetItem::createChildren()
540540

541541
try
542542
{
543-
layerNames = QgsGrass::vectorLayers( mGrassObject.gisdbase(), mGrassObject.location(), mName, name );
543+
layerNames = QgsGrass::vectorLayers( mGrassObject.gisdbase(), mGrassObject.location(), mGrassObject.mapset(), name );
544544
}
545545
catch ( QgsGrass::Exception &e )
546546
{
@@ -611,7 +611,7 @@ QVector<QgsDataItem*> QgsGrassMapsetItem::createChildren()
611611
QString uri = mDirPath + "/" + "cellhd" + "/" + name;
612612
QgsDebugMsg( "uri = " + uri );
613613

614-
QgsGrassObject rasterObject( mGrassObject.gisdbase(), mGrassObject.location(), mName, name, QgsGrassObject::Raster );
614+
QgsGrassObject rasterObject( mGrassObject.gisdbase(), mGrassObject.location(), mGrassObject.mapset(), name, QgsGrassObject::Raster );
615615
if ( objectInImports( rasterObject ) )
616616
{
617617
QgsDebugMsg( "skip currently being imported raster " + name );
@@ -634,7 +634,7 @@ QVector<QgsDataItem*> QgsGrassMapsetItem::createChildren()
634634
QString uri = mDirPath + "/" + "group" + "/" + name;
635635
QgsDebugMsg( "uri = " + uri );
636636

637-
QgsGrassObject rasterObject( mGrassObject.gisdbase(), mGrassObject.location(), mName, name, QgsGrassObject::Group );
637+
QgsGrassObject rasterObject( mGrassObject.gisdbase(), mGrassObject.location(), mGrassObject.mapset(), name, QgsGrassObject::Group );
638638
QgsGrassGroupItem *layer = new QgsGrassGroupItem( this, rasterObject, path, uri );
639639
items.append( layer );
640640
}
@@ -832,7 +832,7 @@ bool QgsGrassMapsetItem::handleDrop( const QMimeData * data, Qt::DropAction )
832832
QgsDebugMsg( "mapsetCrs = " + mapsetCrs.toWkt() );
833833

834834
bool settingsExternal = settings.value( "/GRASS/browser/import/external", true ).toBool();
835-
QgsGrassObject rasterObject( mGrassObject.gisdbase(), mGrassObject.location(), mName, destName, QgsGrassObject::Raster );
835+
QgsGrassObject rasterObject( mGrassObject.gisdbase(), mGrassObject.location(), mGrassObject.mapset(), destName, QgsGrassObject::Raster );
836836
if ( providerCrs.isValid() && mapsetCrs.isValid() && providerCrs == mapsetCrs
837837
&& rasterProvider->name() == "gdal" && settingsExternal )
838838
{
@@ -867,7 +867,7 @@ bool QgsGrassMapsetItem::handleDrop( const QMimeData * data, Qt::DropAction )
867867
else if ( u.layerType == "vector" )
868868
{
869869
QString path = mPath + "/" + "raster" + "/" + u.name;
870-
QgsGrassObject vectorObject( mGrassObject.gisdbase(), mGrassObject.location(), mName, destName, QgsGrassObject::Vector );
870+
QgsGrassObject vectorObject( mGrassObject.gisdbase(), mGrassObject.location(), mGrassObject.mapset(), destName, QgsGrassObject::Vector );
871871
import = new QgsGrassVectorImport( vectorProvider, vectorObject ); // takes provider ownership
872872
}
873873

0 commit comments

Comments
 (0)