Error in the SphericalFromCartesian function

The results of the following function seem wrong to me,

function (::SphericalFromCartesian)(x::AbstractVector)
length(x) == 3 || error("Spherical transform takes a 3D coordinate")

Spherical(hypot(x[1], x[2], x[3]), atan(x[2], x[1]), atan(x[3], hypot(x[1], x[2])))

end

Should the fourth line be changed to this?

function (::SphericalFromCartesian)(x::AbstractVector)
length(x) == 3 || error("Spherical transform takes a 3D coordinate")

Spherical(hypot(x[1], x[2], x[3]), atan(x[2], x[1]), atan(hypot(x[1], x[2]), x[3]))

end

Start by locating SphericalFromCartesian and inspecting the Spherical constructor and related coordinate-transform definitions. Verify which spherical-angle convention the package expects, then compare both formulas against representative 3D coordinates. Done means the function follows the established convention and the relevant behavior is covered by tests.