Динамически добавленное событие команды ImageButton не срабатывает до второго щелчка

Я предполагаю, что проблема может быть вызвана тем, что вы не установили IDs динамически созданных элементов управления. Это очень важно, так как используется в процессе запуска событий обратной отправки. Значение ID для каждого элемента управления должно быть постоянным и не меняться между обратными передачами.

Вы можете попробовать изменить свой метод _LoadGalleryImages() следующим образом:

private void _LoadGalleryImages() {
    PhotoGalleryImageCollection images = PhotoGalleryImages.GetPhotoGalleryImages();

    int imageCtrlCounter = 0;
    foreach(PhotoGalleryImage image in images) {
        ImageButton displayImage = new ImageButton() { ID = String.Format("myDisplayImage{0}", imageCtrlCounter) };
        Panel panel = new Panel();
        panelPhotoContainer.Controls.Add(panel);
        displayImage.ImageUrl = "some URL";

        if(!_IsSettingPrimaryPhotoMode) {
            displayImage.OnClientClick = "showPopup(); return false;";
        }
        else {
            displayImage.Command += new CommandEventHandler(displayImage_Command);
            displayImage.CommandName = "ImageButton" + image.PhotoGalleryImageId.ToString();
            displayImage.CommandArgument = image.PhotoGalleryImageId.ToString();
        }

        panel.Controls.Add(displayImage);
        imageCtrlCounter++;
    }
}

Это должно быть что-то, связанное с начальным подключением события, потому что проблема возникает только в первый раз. Что отличается от того времени, так это то, что _LoadGalleryImages вызывается дважды (один раз в инициализации и один раз в обработчике события нажатия кнопки), поэтому я думаю, что что-то там не очищается, когда вы очищаете панель контейнера и снова вызываете _LoadGalleryImages при нажатии кнопки обработчик.

Почему бы не попробовать эту альтернативу: вызывать LoadImageGalleries только один раз за цикл страницы (в инициализации).

Вместо того, чтобы очищать элементы управления и снова вызывать LoadGalleryImages в той же обратной передаче (в событии нажатия кнопки), при нажатии кнопки вызывается метод, который перебирает элементы управления изображением, которые вы уже создали при вызове LoadGalleryImages, и корректирует их:

1) Удалите onclientclick (очистите его).

2) Прикрепить событие.

@swannee
На самом деле, ваш метод действительно сработал после того, как я подумал об этом. Теперь я вызываю _LoadGalleryImages для каждого OnInit. Я понимаю, что это много повторяющегося кода, который можно объединить.

Новый _LoadGalleryImages

private void _LoadGalleryImages() {
    PhotoGalleryImageCollection images = PhotoGalleryImages.GetPhotoGalleryImages();

    foreach(PhotoGalleryImage image in images) {
        Panel panel = new Panel();
        panelPhotoContainer.Controls.Add(panel);

        ImageButton displayImage = new ImageButton();
        panel.Controls.Add(displayImage);
        displayImage.ID = string.Format("ImageButton{0}", image.PhotoGalleryImageId);
        displayImage.ImageUrl = "Some URL";
        displayImage.AlternateText = displayImage.ToolTip = image.ImageName;

        if(!_IsSettingPrimaryPhotoMode) {
            displayImage.OnClientClick = "showPopup(); return false;";
        }
        else {
            // handles the image button command wireup
            displayImage.Command += new CommandEventHandler(displayImage_Command);
            displayImage.CommandArgument = image.PhotoGalleryImageId.ToString();
        }
    }
}

Я добавил новый метод, как вы предложили, чтобы найти элементы управления, поскольку они уже были созданы в OnInit, и мне просто нужно найти их после нажатия кнопки и очистить OnClientClick.

private void _LoadSelectPrimaryImages() {
    PhotoGalleryImageCollection images = PhotoGalleryImages.GetPhotoGalleryImages();

    foreach(PhotoGalleryImage image in images) {
        Control control = panelPhotoContainer.FindControl(string.Format("ImageButton{0}", image.PhotoGalleryImageId));

        if(control != null) {
            ImageButton displayImage = (ImageButton)control;
            displayImage.OnClientClick = "";
        }
    }
}

У меня также есть кнопка отмены, чтобы вернуть кнопки изображения в то состояние, в котором они были до отображения всплывающего окна.

private void _ResetGalleryImages() {
    PhotoGalleryImageCollection images = PhotoGalleryImages.GetPhotoGalleryImages(_photoGalleryId, false, true);

    foreach(PhotoGalleryImage image in images) {
        Control control = panelPhotoContainer.FindControl(string.Format("ImageButton{0}", image.PhotoGalleryImageId));

        if(control != null) {
            ImageButton displayImage = (ImageButton)control;
            displayImage.ImageUrl = "Original URL";
            displayImage.OnClientClick = "showPopup(); return false;";
        }
    }
}

и две кнопки страницы

protected void btnSetPrimaryPhoto_Click(object sender, EventArgs e) {
    _IsSettingPrimaryPhotoMode = true;
    _LoadSelectPrimaryImages();
}

protected void btnCancelSetPrimaryPhoto_Click(object sender, EventArgs e) {
    _IsSettingPrimaryPhotoMode = false;
    _ResetGalleryImages();
}

Кто-то сказал в ответе ранее... похоже, что ответ был удален... чтобы очистить элементы управления в _LoadGalleryImages, такие как:

private void _LoadGalleryImages() {
    panelPhotoContainer.Controls.Clear();
    PhotoGalleryImageCollection images = PhotoGalleryImages.GetPhotoGalleryImages();

    foreach(PhotoGalleryImage image in images) {
        ImageButton displayImage = new ImageButton();
        Panel panel = new Panel();
        panelPhotoContainer.Controls.Add(panel);
        displayImage.ImageUrl = "some URL";

        if(!_IsSettingPrimaryPhotoMode) {
            displayImage.OnClientClick = "showPopup(); return false;";
        }
        else {
            displayImage.Command += new CommandEventHandler(displayImage_Command);
            displayImage.CommandName = "ImageButton" + image.PhotoGalleryImageId.ToString();
            displayImage.CommandArgument = image.PhotoGalleryImageId.ToString();
        }

        panel.Controls.Add(displayImage);
    }
}

который также работает, но я думаю, что он может быть более неэффективным, чем ваш метод, @swannee. Спасибо!