Change Details

The error handling in `DiffusionRepositoryController::buildNormalContent()` appears to assume that it will handle the futures being resolved via a `FutureIterator`, but then also later calls `resolve()` for each individual future inside of a try/catch (except for the one in `buildBranchList()` which is however itself called in a try/catch). This removes the futures array to avoid resolving the futures outside of a try/catch block, and allows the existing error handling for each piece to take over. Additionally the history query isn't actually used. It looks like it was previously used to display the last user which modified a file however this looks to be removed in D21404. This removes the history query from happening as well as updates `DiffusionBrowseTableView` to remove the parameters for passing this information in, resulting in also updating `DiffusionBrowseController` to no longer need to put this information together.

The error handling descrived in T13666 already exists in `DiffusionRepositoryController` however before it can be used all the queries to different resources are attempted to be resolved outside of a try/catch by use of a `FutureIterator`. The need for resolving all futures up front before processing any of the results should not be necessary, and removing it will allow the existing error handling to kick in if a future fails to resolve to a valid value. Additionally the history query isn't actually used. It looks like it was previously used to display the last user which modified a file however this looks to be removed in D21404. This removes the history query from happening as well as updates `DiffusionBrowseTableView` to remove the parameters for passing this information in, resulting in also updating `DiffusionBrowseController` to no longer need to put this information together.

The error handling in `DiffusionRepositoryController::buildNormalContent()` appears to assume that it will handle the futures being resolved via a `FutureIterator`, but then also later calls `resolve()` for each individual future inside of a try/catch (except for the one in `buildBranchList()` which is however itself called in a try/catch)descrived in T13666 already exists in `DiffusionRepositoryController` however before it can be used all the queries to different resources are attempted to be resolved outside of a try/catch by use of a `FutureIterator`. This removes theThe need for resolving all futures array to avoid resolvup front before processing the futures outside of a try/catch blockany of the results should not be necessary, and removing it will allows the existing error handling for each piecto kick in if a future fails to resolve to take overa valid value. Additionally the history query isn't actually used. It looks like it was previously used to display the last user which modified a file however this looks to be removed in D21404. This removes the history query from happening as well as updates `DiffusionBrowseTableView` to remove the parameters for passing this information in, resulting in also updating `DiffusionBrowseController` to no longer need to put this information together.