Adding an index_toUUID function

[ElliottMess]

No description provided.

[mabafaba]

just FYI we originally had this functionality embedded within each function, then took it out and generalised it to:

cleaninginspectoR/R/inspection_add_data_columns.R

Line 3 in 0e91d88

inspection_add_data_columns<-function(inspection,df,variables){

so for uuids it would be:

inspection_add_data_columns(inspection_output, original_data, "uuid")

Your code looks much safer though (all the checks etc. & using dplyr to join); maybe it can be generalised and put inside inspection_add_data_columns, and then just add a wrapper that's specific to uuids?

We had an intention to keep cleaninginspectoR not kobo specific which made sense to me but i don't have any emotional attachment to that in any case! Do what's useful to you obviously (:

edit: I'm seeing that we never exported that function apparently (?).. not sure (:

[ElliottMess]

Good to know that the function already exists, I missed it indeed (I didn't look for it either). I am modifying my pull request with your suggestion.

I don't think the concept of UUID is unique to kobo either. Most survey and other cases where I can see cleaninginspectoR be used would include some level of unique identifier. You also somehow recognize that by having 2 functions that look for duplicates in UUIDs essentially (:

I guess the issues I was trying to fix was somewhere else though. The inspecting functions return the row number as the index as the unique identifier, which is fine when no unique identifier is inputed, but some functions (including the wrapper inspect_all) do take an UUID variable inputed by the user. If that's the case, in my opinion, the UUID column should be included in the output.