Comment 4 for bug 1211758

Revision history for this message
Aaron Wells (u-aaronw) wrote :

Hi Robert,

While working on the Page Skins feature, I had an idea relating to this. One way we can help verify that all attached File artefacts are ones that the user is allowed to use, is that we could put it in the Pieform filebrowser element. Normally, the best function to put it in would probably be the pieform_element_{$type}_set_attributes() function, but in the case of filebrowser it hacks the normal control flow for Pieforms and does the upload in pieform_element_filebrowser_get_value(). So we'd have to place it in there somewhere if we were to do this.

If the other artefact selectors are using Pieform elements, we may be able to use something similar for them.

Cheers,
Aaron