BUG in set.pluginOption() in spGRASS6

Roger Bivand <Roger.Bivand at nhh.no> writes:

On Fri, 29 Mar 2013, Rainer M Krug wrote:

Hi

using spgrass6_0.7-15 on Ubuntu, there is an error in the function
set.pluginOption() as it always raises an error "logical argument required".

The Check

if (!is.logical(value) || !is.null(value)) stop("logical argument required")

always returns TRUE and therefore always raises the error.

Please try the recent commit to R-forge, spgrass6 module, rspatial
project.

Hi Roger

just installed the one from R-forge and it is working for TRUE and FALSE
- but it is not possible to set it to NULL - is this by design?

As I want tu use FALSE, I am fiine with as it is, but it would be
consistent with the possible values for "plugin" if one could set NULL
as well.

Thanks,

Rainer

Roger

Cheers,

Rainer

On Tue, 2 Apr 2013, Rainer M. Krug wrote:

Roger Bivand <Roger.Bivand at nhh.no> writes:

On Fri, 29 Mar 2013, Rainer M Krug wrote:

Hi

using spgrass6_0.7-15 on Ubuntu, there is an error in the function
set.pluginOption() as it always raises an error "logical argument required".

The Check

if (!is.logical(value) || !is.null(value)) stop("logical argument required")

always returns TRUE and therefore always raises the error.

Please try the recent commit to R-forge, spgrass6 module, rspatial
project.

Hi Roger

just installed the one from R-forge and it is working for TRUE and FALSE
- but it is not possible to set it to NULL - is this by design?

I've reverted to permit NULL as well as TRUE and FALSE as it is the
default value (use plugin if available) in revision 1371.

Roger

As I want tu use FALSE, I am fiine with as it is, but it would be
consistent with the possible values for "plugin" if one could set NULL
as well.

Thanks,

Rainer

Roger

Cheers,

Rainer