Error with multiple covariates in 0.17-8

[tkerwin]

Example code:

require(psych)
require(afex)
data(msq)
aov_ez(data=msq, dv="Extraversion", id = "ID", between = "condition", 
    covariate=c("TOD", "MSQ_Time"), factorize=FALSE)

In afex 0.16-1, this works fine, but in 0.17-8, I get this error:

Error in `[.data.frame`(data, , between, drop = FALSE) : 
  undefined columns selected

It looks like the names of the covariates are being concatenated in a strange way.

[singmann]

Thanks for this report, this is most likely due to the changes in 134afd6. Paging @jonathon-love who made this changes.
Are you willing to remove the bug you introduced (I guess)?

See:

require(psych)
require(afex)
data(msq)
aov_ez(data=msq, dv="Extraversion", id = "ID", between = "condition", 
    covariate=c("TOD", "MSQ_Time"), factorize=FALSE, print.formula = TRUE)
# Formula send to aov_car: Extraversion ~ condition + `TOD+MSQ_Time` + Error(ID)
# Error in `[.data.frame`(data, , between, drop = FALSE) : 
#   undefined columns selected

[jonathon-love]

sure, i'll take a look.

cheers

[singmann]

@jonathon-love Actually hold on. I think I got it, seems really minor.

[jonathon-love]

yup, i think it's as simple as moving line 415:

if (!is.null(covariate)) covariate <- str_c(covariate, collapse = "+")

down after where the covariates are escaped?

covariate <- escape_vars(covariate)

[singmann]

Solved in ccada51.

My change was similar to what you suggested:

if (!is.null(covariate)) {
   covariate <- escape_vars(covariate)
   covariate <- str_c(covariate, collapse = "+")
 }

I will push a new bug-free version to CRAN latest next week. Thanks for reporting the bug.

[singmann]

Fixed version (0.18-0) is on CRAN now: https://cran.r-project.org/web/packages/afex/