[Qgis-developer] Re: [QGIS Commit] r5086 - trunk/qgis/src/gui

[Brendan Morley]

Folks,

Sorry I didn't test for non-provider raster layers as well as my pet
raster type (WMS).

The below change will still disable the Identify tool for non-provider
raster layers, which I believe is still a bug.

I will commit in due course.


Brendan


On Fri, 2006-03-24 at 06:38 +0000, gsherman at qgis.org wrote:
> Author: rblazek
> Date: 2006-03-24 06:38:14 +0000 (Fri, 24 Mar 2006)
> New Revision: 5086
> 
> Modified:
>    trunk/qgis/src/gui/qgisapp.cpp
> Log:
> driver test also
> 
> Modified: trunk/qgis/src/gui/qgisapp.cpp
> ===================================================================
> --- trunk/qgis/src/gui/qgisapp.cpp	2006-03-24 04:53:39 UTC (rev 5085)
> +++ trunk/qgis/src/gui/qgisapp.cpp	2006-03-24 06:38:14 UTC (rev 5086)
> @@ -4958,7 +4958,7 @@
>        {
>          const QgsRasterDataProvider* dprovider = vlayer->getDataProvider();
>          // does provider allow the identify map tool?
> -        if (dprovider->capabilities() & QgsRasterDataProvider::Identify)
> +        if ( dprovider && (dprovider->capabilities() & QgsRasterDataProvider::Identify))
>          {
>            mActionIdentify->setEnabled(TRUE);
>          }