[FIX] cleaner handling of missing dependency

[nbeliy]

All Submissions:

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes: added a test if file referenced by IntendedFor field is indexed (if isempty(info_dest.file_idx)).
Now it produces warning (if verbose) and skips the file:

Warning: IntendedFor file /Storage/beliy/bidsifications/osf/source/sub-01/fmap/sub-01_acq-headMTw_RB1COR.json from sub-01_magnitude1.nii not indexed

Removed the try/catch introduced in #471 as it may hide possible bugs and blame input datasets.

This try/catch can be re-introduced, but I feel it must protect full manage_dependencies function.

Making request draft, as there an alternative implementation of the fix #474.

[Remi-Gau]

closes #470

I have a preference for this one if only because all the tests pass out of the box.

[Remi-Gau]

LGTM.

Thanks @nbeliy