Remove as.zoo.data.frame()

[joshuaulrich]

quantmod::as.zoo.data.frame() overwrites zoo's S3 registry entry when quantmod is loaded. The quantmod method was added in 2007, and the zoo method was not added until 2010. The package that owns the class should define the conversion method, so the as.zoo.*() methods should be in zoo.

Removing quantmod::as.zoo.data.frame() may break user code, because it converts rownames to Date before using them as the index for the zoo object, and the zoo method only uses an integer index by default.

The migration was started in 9343d9a by by printing a message the first time the quantmod method is called. Throw a warning so the method can be removed in the next major release.

Related to #315.

[ctmaher]

Any progress on this? I get the "Registered S3 method overwritten by 'quantmod'" message when I load my package even though I don't use quantmod as a dependency in my package (nor do I use zoo), but one of my dependencies uses zoo. I assume that at some level further upstream there is a quantmod dependency. There is no effect on my code at all, but the message is undesirable. Thanks!

[joshuaulrich]

@ctmaher A new version of quantmod is going to CRAN soon. It will include a warning the first time quantmod:::as.zoo.data.frame() is called, and give users the code to use instead. I'll remove this method in a release that is at least 6 months from the upcoming release. I know that seems like a long time, but I want to give users some time to migrate their code before removing the function and potentially breaking their code.