From ec035622a96da724d344cc88a2e5af6f399ea758 Mon Sep 17 00:00:00 2001
From: Delaigue Olivier <olivier.delaigue@irstea.priv>
Date: Tue, 29 Jan 2019 13:36:52 +0100
Subject: [PATCH] v1.1.2.36 CLEAN: change the message and the place of the
 check on obs in CreateInputsCrit

---
 DESCRIPTION          |  2 +-
 NEWS.rmd             |  2 +-
 R/CreateInputsCrit.R | 12 ++++++------
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/DESCRIPTION b/DESCRIPTION
index 4a9e3e20..b3347eac 100644
--- a/DESCRIPTION
+++ b/DESCRIPTION
@@ -1,7 +1,7 @@
 Package: airGR
 Type: Package
 Title: Suite of GR Hydrological Models for Precipitation-Runoff Modelling
-Version: 1.1.2.35
+Version: 1.1.2.36
 Date: 2019-01-29
 Authors@R: c(
   person("Laurent", "Coron", role = c("aut", "trl"), comment = c(ORCID = "0000-0002-1503-6204")),
diff --git a/NEWS.rmd b/NEWS.rmd
index 254dcdaa..971c3b30 100644
--- a/NEWS.rmd
+++ b/NEWS.rmd
@@ -13,7 +13,7 @@ output:
 
 
 
-### 1.1.2.35 Release Notes (2019-01-29) 
+### 1.1.2.36 Release Notes (2019-01-29) 
 
 
 
diff --git a/R/CreateInputsCrit.R b/R/CreateInputsCrit.R
index d45f4a1c..2c59f625 100644
--- a/R/CreateInputsCrit.R
+++ b/R/CreateInputsCrit.R
@@ -134,6 +134,12 @@ CreateInputsCrit <- function(FUN_CRIT,
       return(NULL)
     }
     
+    ## check 'obs'
+    if (!is.vector(iListArgs2$obs) | length(iListArgs2$obs) != LLL | !is.numeric(iListArgs2$obs)) {
+      stop(sprintf("'obs' must be a (list of) vector(s) of numeric values of length %i \n", LLL), call. = FALSE)
+      return(NULL)
+    }
+    
     ## check 'BoolCrit'
     if (is.null(iListArgs2$BoolCrit)) {
       iListArgs2$BoolCrit <- rep(TRUE, length(iListArgs2$obs))
@@ -147,12 +153,6 @@ CreateInputsCrit <- function(FUN_CRIT,
       return(NULL)
     }
     
-    ## check 'obs'
-    if (!is.vector(iListArgs2$obs) | length(iListArgs2$obs) != LLL | !is.numeric(iListArgs2$obs)) {
-      stop("'obs' must be a (list of) vector(s) of numeric values \n", call. = FALSE)
-      return(NULL)
-    }
-    
     ## check 'varObs'
     if (!is.vector(iListArgs2$varObs) | length(iListArgs2$varObs) != 1 | !is.character(iListArgs2$varObs) | !all(iListArgs2$varObs %in% inVarObs)) {
       stop(msgVarObs, call. = FALSE)
-- 
GitLab