Skip to content

Commit

Permalink
Merge pull request #140 from RSGInc/139-small-bug-in-hts_prep_variable
Browse files Browse the repository at this point in the history
changing != to !%in% where needed
  • Loading branch information
james-clark-rsg authored Mar 4, 2024
2 parents f4666a3 + 4b9c606 commit 67b95f7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions R/hts_prep_variable.R
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ hts_prep_variable = function(summarize_var = NULL,
summarize_var = variables_dt[shared_name == summarize_var, variable]

for (i in 1:length(summarize_var)) {
if (var_dt[, class(get(summarize_var[i]))] != "integer") {
if (var_dt[, !"integer" %in% class(get(summarize_var[i]))]) {
message("Checkbox variables must have integer values")

stop()
Expand Down Expand Up @@ -273,7 +273,7 @@ hts_prep_variable = function(summarize_var = NULL,
var = variables_dt[shared_name == var, variable][1]
}

if (byvar_is_shared & byvar_table[, class(get(var))] != "integer") {
if (byvar_is_shared & !"integer" %in% byvar_table[, class(get(var))]) {
message("Checkbox variables must have integer values")

stop()
Expand Down

0 comments on commit 67b95f7

Please sign in to comment.