jsonを使ってjQueryで機能するモーダルウィンドウを作りたい

Question

初めまして、以下のことを質問させてください。
jsonでモーダルウィンドウのボタンとモーダルウィンドウのhtmlを書き出しするのはできたのですが、
ボタンをクリックしてもモーダルウィンドウの中に描画したhtml(文字や画像)が出なくて困っています。
jsonでの出力ではなく、htmlにjsonの部分も全部書いた場合はモーダルウィンドウはうまく機能します。

どうすればうまくいくか試行錯誤しています。
モーダルウィンドウのjsに、例えばA会社のボタンをクリックしたら、A会社のidnameを取得するというようにjsを書き換えたらうまくいくのでは？と思い以下のことを試しています。

jsonの"idname"を
jsのvar link = self.attr(users.idname); で取得しようとしました。
うまく行きません。
jsonを読み込ませず、htmlとjsのみにした場合は、
var link = self.attr(users.idname);のところは、
var link = self.attr('href'); になります。

jsはunderscore.jsも使っています。

お忙しいところすみません、是非よろしくお願いいたします。

＜jsonの部分＞

{"users": [	
	
{
	"title":"A株式会社",
	"cat":"website",
	"filename":"a-company",
	"idname":"#modal1",
	"url":"http://a-company.co.jp/"
},
{
	"title":"B株式会社",
	"cat":"website",
	"filename":"b-company",
	"idname":"#modal2",
	"url":"http://a-company.co.jp/"
}	
  ]	
}

---

＜jsの部分＞

(function($){

  $(document).ready(function(){
    $.ajax({
      dataType: "json",
      url: "works.json",
      success: function(data) {
        // バインドするオブジェクト定義
        var users = data.users;
        // テンプレートを取得
        var template = $("#external-template").text();
        // テンプレートを定義
        var compiled = _.map(users, function(user) {
          return _.template(template, user);
        });
        outputResults(users);
        outputResults2(users);
        simpleModalWindow(users);
      }
    });
  });
  
（ここには、function outputResults(users);
　  function outputResults2(users); でhtmlにボタンとモーダルウィンドウのhtml出力　を記述）

  function simpleModalWindow(users){

    var sp = 500; //アニメーション速度
    var win = $(window);
    var body = $('body');
    var bg = $('<div id="modal-bg"></div>');
    bg.css('opacity', '0');

    //モーダルウィンドウ表示クリックイベント
    $(document).on('click', '.modal-open', function(){
      var py = win.scrollTop();
      var wh = win.height();
      var self = $(this);
      var link = self.attr(users.idname);
      var check = link.match(/^#.+/);
      var mWin = $('<div id="modal-win"><div id="modal-win-inner"></div></div>');
      var mInner = mWin.find('#modal-win-inner');
      mInner.css('opacity', '0');
      body.append(mWin);
      mWin.prepend(bg);
      if(!check){
        mInner.append('<img src="' + link + '" alt="" />');
        var img = mWin.find('img');
        img.on('load', function(){
          view(img);
        });
      }
      else {
        var contents = $(link);
        mInner.append(contents);
        contents.css({display: 'block', zIndex: '20001'});
        view(contents);
      }
      function view(a_elm){
        var w = a_elm.outerWidth();
        var h = a_elm.outerHeight();
        var mt = (wh - h) / 2 + py;
        bg.animate({opacity: '.75'}, sp);
        mWin.css('top', mt + 'px');
        mInner.css({width: w, height: h}).animate({opacity: '1'}, sp);
      }
      return false;
    });

    //モーダルウィンドウ内要素変更クリックイベント
    $(document).on('click', '.modal-move', function(){
      var py = win.scrollTop();
      var wh = win.height();
      var self = $(this);
      var link = self.attr(users.idname);
      var check = link.match(/^#.+/i);
      var mWin = $('#modal-win');
      var mInner = mWin.find('#modal-win-inner');
      if(check){
        mInner.animate({opacity: '0'}, sp, function(){
          var nowContents = $(this).children();
          body.append(nowContents);
          nowContents.hide();
          var contents = $(link);
          mInner.append(contents);
          contents.css({display: 'block', zIndex: '20001'});
          var w = contents.outerWidth();
          var h = contents.outerHeight();
          var mt = (wh - h) / 2 + py;
          bg.animate({opacity: '.75'}, sp);
          mWin.css('top', mt + 'px');
          mInner.css({width: w, height: h}).animate({opacity: '1'}, sp);
        });
      }
      return false;
    });

    //モーダルウィンドウクローズクリックイベント
    $(document).on('click', '#modal-bg, .modal-close', function(){
      var mWin = $('#modal-win');
      var mInner = mWin.find('#modal-win-inner');
      var contents = mInner.children();
      mInner.animate({opacity: '0'}, sp, function(){
        if(contents.attr("id")){
          body.append(contents);
          contents.hide();
        }
        mWin.remove();
      });
      bg.animate({opacity: '0'}, sp);
      return false;
    });

  };

})(jQuery);

Answer 1

var link = self.attr(users.idname);

このusersというのはajaxのコールバック内のsimpleModalWindow(users);で送られてきているユーザー(会社？)情報ですよね。
おそらくusersは配列だと思いますのでアクセスするなら

var link = self.attr(users[i].idname);

のようにしないといけないのではないでしょうか？

Comments

[nabisuke78]

ご回答ありがとう御座います。
ご回答のようにして、var i = 1;も入れたらそこのエラーは消えました。
でも、
var check = link.match(/^#.+/); の部分で、
Cannot read property 'match' of undefined とエラーが出てしまいました。

[nabisuke78]

このエラーは、要素が入っていない場合に起きるそうなので、users[i].idnameには要素が入っていないという事になりますね。
別の方法考えてみます。

[nabisuke78]

alert(users[i].idname);
で呼び出したら取得してました！！
なので、
var link = self.attr(users.idname);
のself.attrの部分を他の記述にしたらうまくいくかもしれないです。

Answer 2

すみません、jsonだと描画されたのをソースに直接描画できないので、htmlに直打ちして対応しました。
ご回答いただいたのに解決できなくて申し訳ないです。