Skip to content

Commit e82b934

Browse files
committed
remove na-checking logic
1 parent 626e55e commit e82b934

File tree

1 file changed

+2
-12
lines changed

1 file changed

+2
-12
lines changed

facebook/delphiFacebook/R/variables.R

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -653,15 +653,10 @@ code_vaccines <- function(input_data, wave) {
653653
# introduced in Wave 11
654654
vaccine_barriers <- coalesce(input_data$V15a, input_data$V15b)
655655

656-
# If the entire column is NA, ifelse() results in a logical vector, not a
657-
# character vector, which confuses split_options; since the result should be
658-
# NA anyway
659656
vaccine_barriers <- as.character(
660657
ifelse(vaccine_barriers == "13", NA_character_, vaccine_barriers)
661658
)
662-
if (any(!is.na(vaccine_barriers))) {
663-
vaccine_barriers <- split_options(vaccine_barriers)
664-
}
659+
vaccine_barriers <- split_options(vaccine_barriers)
665660

666661
input_data$v_vaccine_barrier_eligible <- is_selected(vaccine_barriers, "1")
667662
input_data$v_vaccine_barrier_no_appointments <- is_selected(vaccine_barriers, "2")
@@ -773,15 +768,10 @@ code_vaccines <- function(input_data, wave) {
773768

774769
if ( "V15b" %in% names(input_data) ) {
775770
# introduced in Wave 11
776-
# If the entire column is NA, ifelse() results in a logical vector, not a
777-
# character vector, which confuses split_options; since the result should be
778-
# NA anyway
779771
vaccine_barriers <- as.character(
780772
ifelse(input_data$V15b == "13", NA, input_data$V15b)
781773
)
782-
if (any(!is.na(vaccine_barriers))) {
783-
vaccine_barriers <- split_options(vaccine_barriers)
784-
}
774+
vaccine_barriers <- split_options(vaccine_barriers)
785775

786776
input_data$v_vaccine_barrier_eligible_tried <- is_selected(vaccine_barriers, "1")
787777
input_data$v_vaccine_barrier_no_appointments_tried <- is_selected(vaccine_barriers, "2")

0 commit comments

Comments
 (0)