How to fill link the column in FormEdit and Edit action?

sblb 1,166

Hi,
My app : blazor wasm aps.net core5.

First of all I tried, I will try to explain the configuration of my application is : DataGridView with several column of which ECR & ECO

The column ECR is fill when the user click on AddEcr. Thanks to @AgaveJoe the number is incremented with the current year. it's ok.
We can acces to ECO column when we click on Edit button. I want that the user can take or not the number ECO xxx/YY (with the same format that ECR).

The button Update correspond to the Edit.razor this means populate all data in table from database.

Edit.razor

 <FormEdit ButtonText="Update" dev="dev"  
       OnValidSubmit="@EditDeveloper" />  
      
 @code {  
      
     [Parameter] public int developerId { get; set; }  
     Developer dev = new Developer();  
      
      
     protected async override Task OnParametersSetAsync()  
     {  
         dev = await http.GetFromJsonAsync<Developer>($"api/developer/{developerId}");  
     }  
      
     async Task EditDeveloper()  
     {  
         await http.PutAsJsonAsync("api/developer", dev);  
         await js.InvokeVoidAsync("alert", $"Updated Successfully!");  
         uriHelper.NavigateTo("developer");  
                             
     }

Now the button Number ECO allows to user take the number .

The definition of the button Number ECO is in FormEdit.razor

 <EditForm Model="@dev" OnValidSubmit="@OnValidSubmit">  
     <DataAnnotationsValidator />  
      
      
  <button type="submit" @onclick="@OnSubmitEcoNumber" >Submit</button>  
      
 </EditForm>  
      
 @code {  
  [Parameter] public Developer dev { get; set; }  
  [Parameter] public EventCallback OnValidSubmit { get; set; }  
          
     Developer[] developers { get; set; }   
 protected void OnSubmitEcoNumber()  
 {  
   //It's here that I've to call the method   
 }  
   }

My question is : how to do the link between the button Number ECO (I think I have to put check box) with the Edit.razor to fill the column ECO?

the api is define below to fill the ECO number with a good format

  [HttpPost]  
        public async Task<IActionResult> Post(Developer developer)  
        {  
            YearCountECO deveco = await IncrementECODeveloperIdAsync();  
  
            developer.YearECO = deveco.YearECO;  
            developer.CountECO = deveco.CountECO;  
  
            YearCount dev = await IncrementDeveloperIdAsync();  
  
            developer.Year = dev.Year;  
            developer.Count = dev.Count;  
  
            _context.Add(developer);  
            await _context.SaveChangesAsync();  
            return Ok(developer.Id);  
        }        
    
  
 private async Task<YearCountECO> IncrementECODeveloperIdAsync()  
        {  
            if (!await _context.Developers.AnyAsync(d => d.Year == DateTime.Now.Year))  
            {  
                return new YearCountECO() { CountECO = 1, YearECO= DateTime.Now.Year };  
            }  
  
            int max = await _context.Developers.Where(d => d.Year == DateTime.Now.Year).MaxAsync(d => d.Count) + 1;  
            return new YearCountECO() { CountECO = max, YearECO = DateTime.Now.Year };  
        }

AgaveJoe 26,201 Reputation points

2022-09-24T11:46:17.357+00:00

My question is : how to do the link between the button Number ECO (I think I have to put check box) with the Edit.razor to fill the column ECO?

I'm very confused by your logic. and the question. The code clearly generates and inserts the ERC and ECO values into the Developer table when the Post method is invoked. No where in the code are you calling this Post method. There's a Put but you did not share that code!

You did not share how the DataGridView is populated. We have no idea how the ECO column is related to the dataset that populates the DataGridView or why you are unable to populate the DataGridView with ERC or ECO values.

Similar to your other threads, I think you're created a design that will not solve the actual problem. I can't read you mind so I have no idea what you're trying to do. Can you explain the use case rather than the solution?

Is the problem the DataGridView is not refreshed after creating the ERC and ECO values?????

11 answers

sblb 1,166 Reputation points

2022-09-25T09:23:09.777+00:00
The code clearly generates and inserts the ERC and ECO values into the Developer table when the Post method is invoked.

I agree with you. Currently the column ECO is fill when I create a new row. But I don't want the value of ECO to be created when a new one is created.

Create.razor

@page "/developer/create" @inject HttpClient http @inject NavigationManager uriHelper <FormCreate ButtonText="Create ECR" dev="@dev" OnValidSubmit="@CreateDeveloper"/> @code { Developer dev = new Developer(); async Task CreateDeveloper() { await http.PostAsJsonAsync("api/developer", dev); uriHelper.NavigateTo("developer"); } }

Edit.razor

@page "/developer/edit/{developerId:int}" @inject HttpClient http @inject NavigationManager uriHelper @inject IJSRuntime js <FormEdit ButtonText="Update" dev="dev" OnValidSubmit="@EditDeveloper" /> @code { [Parameter] public int developerId { get; set; } Developer dev = new Developer(); protected async override Task OnParametersSetAsync() { dev = await http.GetFromJsonAsync<Developer>($"api/developer/{developerId}"); } async Task EditDeveloper() { await http.PutAsJsonAsync("api/developer", dev); await js.InvokeVoidAsync("alert", $"Updated Successfully!"); uriHelper.NavigateTo("developer"); } }

FetchData.razor

..... <RadzenDataGridColumn TItem="Developer" Property="ECO" Title="ECO"> <Template Context="developers"> <span style='color:black'> @developers.ECO</span> </Template> ..... @code{ .... Developer[] developers { get; set; } protected override async Task OnInitializedAsync() { developers = await client.GetFromJsonAsync<Developer[]>("api/developer"); } .... }

Can you explain the use case rather than the solution?

I would like to create the ECO value in the Edit page and not in the create page.
I want to give the user the option to create or not create an ECO number in the Edit page for example with a button or a checkbox.

My question: how do I create the ECO number in the Edit page?
Please sign in to rate this answer.
AgaveJoe 26,201 Reputation points

2022-09-25T11:28:56.59+00:00

I asked for the use case not the solution. The reason I asked for the use case is your solutions are typically nonsensical. If the community writes your code, again, you'll certainly complain the approach does not work as expected.

I do have a few questions. The ECO is a system generated value based on the current state of the Developer table. Your expectation is the user magically knows the state of the Developer table and how to enter a valid value? The main purpose of the system generated value is to eliminate duplicates and eliminate the user from guessing what the next unique value is. Is the intention of this design to irritate the user?

Secondly, if you already have a unique system generated value, why are you creating a second unique system generated value in the same record? this approach makes little sense.

My question: how do I create the ECO number in the Edit page?

You already have the code to get the next ECO value so I'm very confused why you are unable to move forward.
Sign in to comment
sblb 1,166 Reputation points

2022-09-25T15:36:46.06+00:00

I asked for the use case not the solution.

I try to explain the use case with some pictures.

1- Click on Add ECR which open a new page.

2-Click on create ECR. The new row is created

3- New line created. Open the edit mode

4-Navigate into Edit mode

5-The user decide to take Eco Number

6- Result in ECO Column
Please sign in to rate this answer.
AgaveJoe 26,201 Reputation points

2022-09-25T19:09:55.003+00:00

You shared the solution again not the use case. I give up.

sblb 1,166 Reputation points

2022-09-26T06:23:20.457+00:00

I'm sorry but I don't understand what you mean by use case.

You shared the solution

??
Sign in to comment
sblb 1,166 Reputation points

2022-09-26T10:03:57.697+00:00

the use case can be explain as below :
_the user create a new ECR in page create and a ECR number is taken.
_ the ECO number is not in page create, it is only in mode edit.
_ the user can choose to take or not the ECO number by validated the checkbox;
_ In a row there may be an ECR number without an ECO number.
_ The format ECO must be xxx/YY ; xxx is integer which is incremented for each row and YY two digits of the current year.
Please sign in to rate this answer.
AgaveJoe 26,201 Reputation points

2022-09-26T11:35:13.48+00:00

You continue to explain the solution not the use case and you did not answer any of my previous questions.

The format ECO must be xxx/YY ; xxx is integer which is incremented for each row and YY two digits of the current year.

You expect the application users to query the database to find the next count value and enter a one when the year changes and no ECO numbers exist? Your solution does not make sense to me and I have no intention of writing your proof of concept. Maybe someone else on the forum can help you or perhaps you can write your own code.
Sign in to comment
sblb 1,166 Reputation points

2022-09-26T16:10:26.807+00:00
I follow up your recommendation here : WebApiSqlite to do the ECR column.
In your github you didn't use FormattedId; So I have adapted the code

public string ECR { get { return $"{Count.ToString().PadLeft(3, '0')}/{Year}"; } set { } }

If I've to use FormatId I've to add return ECR = FormatId , right?

I did not see any code that populates ECO in this thread

I invoke Put method to populate ECO, no?

<FormEdit ButtonText="Update" dev="dev" OnValidSubmit="@EditDeveloper" /> @code { [Parameter] public int developerId { get; set; } Developer dev = new Developer(); protected async override Task OnParametersSetAsync() { dev = await http.GetFromJsonAsync<Developer>($"api/developer/{developerId}"); } async Task EditDeveloper() { await http.PutAsJsonAsync("api/developer", dev); await js.InvokeVoidAsync("alert", $"Updated Successfully!"); uriHelper.NavigateTo("developer"); }
Please sign in to rate this answer.
AgaveJoe 26,201 Reputation points

2022-09-26T16:58:16.693+00:00

I've explained time and time again not use an empty set with an entity property. I even provided test code that shows what bad things can happen yet the ECR has an empty set.

If I've to use FormatId I've to add return ECR = FormatId , right?

I think you completely missed the point. There is no logical reason to have an ECO column in a table since the ECO value is combination of the count and the year. You've created redundant data and extra logic to write the ECO value.

I invoke Put method to populate ECO, no?

Your Put method does not set the ECO value! It sets the count and the year. I have no idea if you are using a ViewModel or the Entity with the empty set in the UI but your Put action accepts whatever value is in ECO property submitted form the client and saves it in the column.

Seriously, what do you want from this community? You don't follow advice or recommendations. And as far as I can tell you do not run your code through the debugger.
Sign in to comment

sblb 1,166

You expect the application users to query the database to find the next count value and enter a one when the year changes and no ECO numbers exist?

Are you doing it on purpose?

you can write your own code.

According to you I already have the solution, right?

The code is :

<EditForm Model="@dev" OnValidSubmit="@OnValidSubmit">  
    <DataAnnotationsValidator />  
  
 <RadzenCheckBox @bind-Value=@checkBox1Value Disabled="value:false" Name="CheckBox1" TValue="bool" Change=@(args => OnChange(args)) />  
                                            <RadzenLabel Text="Take ECO Number" Component="CheckBox1" Style="margin-left: 8px; vertical-align: middle;" />  
                                            @if(checkBox1Value == true)  
                                            {  
                                               dev.ECO = $"{dev.CountECO.ToString().PadLeft(3, '0')}/{dev.YearECO}";  
                                                
                                            }  
  
</EditForm>  
  
  
  
<FormEdit ButtonText="Update" dev="dev"  
      OnValidSubmit="@EditDeveloper" />  
  
@code {  
  
    [Parameter] public int developerId { get; set; }  
    Developer dev = new Developer();  
  
  
    protected async override Task OnParametersSetAsync()  
    {  
        dev = await http.GetFromJsonAsync<Developer>($"api/developer/{developerId}");  
    }  
  
    async Task EditDeveloper()  
    {  
        await http.PutAsJsonAsync("api/developer", dev);  
        await js.InvokeVoidAsync("alert", $"Updated Successfully!");  
        uriHelper.NavigateTo("developer");  
                         
    }

api controller

        [HttpPut]  
        public async Task<IActionResult> Put(Developer developer)  
        {  
            YearCountECO deveco = await IncrementECODeveloperIdAsync();  
  
            developer.YearECO = deveco.YearECO;  
            developer.CountECO = deveco.CountECO;  
  
            _context.Entry(developer).State = EntityState.Modified;  
            await _context.SaveChangesAsync();  
            return NoContent();  
        }  
  
        //Take a number ECO  
        private async Task<YearCountECO> IncrementECODeveloperIdAsync()  
        {  
            if (!await _context.Developers.AnyAsync(deco => deco.YearECO == DateTime.Now.Year))  
            {  
                return new YearCountECO() { CountECO = 1, YearECO = DateTime.Now.Year };  
            }  
  
            int max = await _context.Developers  
                                    .Where(deco => deco.YearECO == DateTime.Now.Year).MaxAsync(deco => deco.CountECO) + 1;  
            return new YearCountECO() { CountECO = max, YearECO = DateTime.Now.Year };  
  
        }

With this code I obtained

As you can see in row 1 and 3 the user decides not to take an ECO number.
In line 2 & 4 the user decide to take an Eco number when checkboxValue1 = true.

But as you can see the count is not good.

I don't understand why I obtained the ECO number in row 2 : 001/2022 then CountECO = 2

AgaveJoe 26,201 Reputation points

2022-09-26T12:14:33.027+00:00

According to you I already have the solution, right?

Yes. you already have the code to programmatically generate the number.

I don't understand why I obtained the ECO number in row 2 : 001/2022 then CountECO = 2

Now you're asking a new question! Try using the Visual Studio debugger to validate your code.

How was the 001/2022 value input? Did the user input the value or was the value system generated? When was the value entered? If it was the first value then then code is correct.
sblb 1,166 Reputation points

2022-09-26T12:35:02.397+00:00

How was the 001/2022 value input? Did the user input the value or was the value system generated?

The value was generated by the system : api + EditForm with the condition on the checkboxValue = true (clicked by the user see a picture below)

If it was the first value then then code is correct.

When I open the Edit mode for the first time + checkboxvalue = true the result is : 000/0
I have to reopen the edit mode + checkbox = true to obtain 001/2022
AgaveJoe 26,201 Reputation points

2022-09-26T15:26:39.307+00:00
If you re-read your previous threads on this subject, I specifically recommended NOT storing XXX/YY in the database. My recommendation is and has always been to use a property when displaying the ECO or ECR values.

[NotMapped] public string FormattedId { get { return $"{Count.ToString().PadLeft(3,'0')}/{Year}"; } }

I even wrote a sample CRUD project and committed it to Github. Yet, you decided to ignore this advice and write the ECO value to the database. Not surprisingly this part of the code is not working. I did not see any code that populates ECO in this thread so I have no idea what you're doing but clearly your logic is not working.

Please set a break point and debug your code.