Modal Resolution Not Using Locals

[micah-williamson]

If a modal resolution contains another resolution from the same resolve block it will fail. This is not working as expected (like ui-router).

Steps to reproduce:

• Create a Controller called MainCtrl that injects the $modal service with a scope method called openModal()
• Create a view MainCtrl is controller. Create a button that, when clicked, calls openModal()
• In the openModal() method use the $modal service to call $modal.open()
• In the $modal.open call add two resolutions. The second resolution should attempt to use the first resolution.

angular.module('app', ['ui.bootstrap'])
  .controller('MainCtrl', function($scope, $modal) {

    $scope.openModal = function() {
      $modal.open({
        template: 'hello'
      });
    };

    $scope.openModal2 = function() {
      $modal.open({
        template: 'hello',
        resolve: {
          r1: function() {
            return 'hello';
          },
          // Will fail
          r2: function(r1) {
            return r1 + ' world!';
          }
        }
      });
    };

  });

JSBin: http://jsbin.com/riyijuqolu/1/edit?js,console,output

Please address.

[BobbieBarker]

I did a look over on how our resolve block is handled in modals and compared it to how ui-router handles their resolve block. Our resolve in comparison is a very simple implementation where all the promises are pushed into an array and $q.all is used. Which is where I think the break down that this issue high lights begins. For comparison if you look at resolve.js from ui-router
https://github.com/angular-ui/ui-router/blob/master/src/resolve.js
You'll see a dramatically more complex implementation. On thate note, I don't think there is a bug here, I just don't think our modal was written to handle this use case.

[micah-williamson]

@RobJacobs @rvanbaalen

I'm not opening a modal from a modal...

[micah-williamson]

@BobbieBarker Do you think this should be updated? Is the expected behavior to work like ui-router resolve?

[BobbieBarker]

Looking at the code I don't think the expected behavior is to have an implementation like ui-routers, but also looking at ui-router's implementation I can see where an enhancement could certainly be made.

[wesleycho]

The UI Router implementation is quite complex compared to ours. We could copy/paste that code, but that would be duplicate code - it may end up being the best solution though. UI Router is undergoing a large refactor right now in TypeScript for the 1.0 release and potential abstraction to decouple it from Angular itself (so it can be used in Angular 2, or in any other application).