How to set ViewModel Properties from DataGrid Selected Row using Caliburn Micro

Question

I am working on one WPF (MVVM) Application with Caliburn Micro, I went by the standard wordings that in properties should reside in VM, In My App I've 3 textboxs and one datagrid, What I want is that whenever use selects any row the Textboxs must show the selected values (Single Selection Only !), Textboxes will show different data based on Columns. I've model also for the same structure, The problem is I've set the x:name of the control as property as per naming conventions of CM, now the selection is taken from the model so How can I set the local VM property from the Model property.

My Model :

public class PackageModel
{
    public int id { get; set; }
    public string sessionName { get; set; }
    public int sessionInMins { get; set; }
    public int sessionAmount { get; set; }
    public bool isActive { get; set; }
}

My ViewModel :

private string _packName;
    public string PackageName
    {
        get { return _packName; }
        set
        {
            _packName = value;
            NotifyOfPropertyChange(() => PackageName);
        }
    }

    private int _amount;
    public int Amount
    {
        get { return _amount; }
        set {
            _amount = value;
            NotifyOfPropertyChange(() => Amount);
        }
    }

    private int _mins;
    public int Mins
    {
        get { return _mins; }
        set {
            _mins = value;
            NotifyOfPropertyChange(() => Mins);
        }
    }

    private bool _isActive;
    public bool IsPackageActive
    {
        get { return _isActive; }
        set {
            _isActive = value;
            NotifyOfPropertyChange(() => IsPackageActive);
        }
    }

    private List _packageList;
    public List PackageList
    {
        get { return _packageList; }
        set
        {
            _packageList = value;
            NotifyOfPropertyChange(() => PackageList);
        }
    }

    private PackageModel _selectedPackage;
    public PackageModel SelectedPackage
    {
        get
        {
            return _selectedPackage;
        }
        set
        {
                _selectedPackage = value;
                NotifyOfPropertyChange(() => SelectedPackage);
        }
        }
    }

My View :

Please suggest if I am following any wrong standard or wrong way of setting properties.

Kaspar · Accepted Answer

These is not Calibro, but you said that it is ok.

I remove the property sessionInMins and sessionAmout to make the answer shorter, you can add it in similar way as other property.

First I add INotifyPropertyChanged to Model, I also make the PascalCase for property (upper letter). PascalCase is not needed but it is most common convention.

public class PackageModel : INotifyPropertyChanged
    {
        private int _Id;
        public int Id
        {
            get { return _Id; }
            set
            {
                if (value != _Id)
                {
                    _Id = value;
                    NotifyPropertyChanged();
                }
            }
        }

        private string _SessionName;
        public string SessionName
        {
            get { return _SessionName; }
            set
            {
                if (value != _SessionName)
                {
                    _SessionName = value;
                    NotifyPropertyChanged();
                }
            }
        }

        private bool _IsActive;
        public bool IsActive
        {
            get { return _IsActive; }
            set
            {
                if (value != _IsActive)
                {
                    _IsActive = value;
                    NotifyPropertyChanged();
                }
            }
        }


        public event PropertyChangedEventHandler PropertyChanged;
        private void NotifyPropertyChanged([CallerMemberName] String propertyName = "")
        {
            PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
        }
    }

I Modify VM only to have List of Packages and SelectedPackage, I remove List from name Packages and make it plural addins's. Also make some defualt constructor to fill developer List. As you also suggest, the additional property is not needed right now, just use SelectedPackage later on.

    public class MainViewModel : INotifyPropertyChanged
    {
        public MainViewModel()
        {
            Packages = new List()
            {
                new PackageModel()
                {
                    Id = 1, SessionName="Session1", IsActive = true
                },
                new PackageModel()
                {
                    Id = 2, SessionName="Session2", IsActive = false
                }
            };
        }

        private PackageModel _SelectedPackage;
        public PackageModel SelectedPackage
        {
            get { return _SelectedPackage; }
            set
            {
                if (value != _SelectedPackage)
                {
                    _SelectedPackage = value;
                    NotifyPropertyChanged();
                }
            }
        }

        private List _Packages;
        public List Packages
        {
            get { return _Packages; }
            set
            {
                if (value != _Packages)
                {
                    _Packages = value;
                    NotifyPropertyChanged();
                }
            }
        }


        public event PropertyChangedEventHandler PropertyChanged;
        private void NotifyPropertyChanged([CallerMemberName] String propertyName = "")
        {
            PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
        }
    }

And also add the View. Do manual bindings. When you change the item in the grid the textbox and checkbox will update automatic, also if you change values in textbox or checkbox it will update the DataGrid as well.

Let me know if solution works for you!

How to set ViewModel Properties from DataGrid Selected Row using Caliburn Micro

Answers (2)

Related Questions