Remove restock everywhere

[mw3155]

User description

Fixes #5

Remove RestockThreshold and MaxStockThreshold properties and references from the project.

• Models: Remove RestockThreshold and MaxStockThreshold properties from CatalogItem.cs.
• Controllers: Remove references to RestockThreshold and MaxStockThreshold in the Create and Edit actions in CatalogController.cs.
• Views: Remove display and form fields for RestockThreshold and MaxStockThreshold in CatalogTable.cshtml, Create.cshtml, Edit.cshtml, and Details.cshtml.
• Infrastructure: Remove references to RestockThreshold and MaxStockThreshold in CatalogDBInitializer.cs.
• Services: Remove references to RestockThreshold and MaxStockThreshold in CatalogService.cs and CatalogServiceMock.cs.

---

For more details, open the Copilot Workspace session.

---

PR Type

enhancement

---

Description

• Removed RestockThreshold and MaxStockThreshold properties from the CatalogItem model.
• Updated CatalogController to remove references to RestockThreshold and MaxStockThreshold in Create and Edit actions.
• Modified views (CatalogTable.cshtml, Create.cshtml, Edit.cshtml, Details.cshtml) to remove display and form fields for RestockThreshold and MaxStockThreshold.
• Updated CatalogDBInitializer to remove processing of RestockThreshold and MaxStockThreshold from CSV files.
• Minor formatting changes in CatalogService.cs and CatalogServiceMock.cs.

---

Changes walkthrough 📝

	Relevant files
Enhancement	7 files CatalogController.cs Remove restock properties from CatalogController actions  eShopOnWeb/eShop.MVC/Controllers/CatalogController.cs Removed RestockThreshold and MaxStockThreshold from Create and Edit actions. +2/-2      CatalogItem.cs Remove restock properties from CatalogItem model                  eShopOnWeb/eShop.MVC/Models/CatalogItem.cs Removed RestockThreshold and MaxStockThreshold properties. +1/-9      CatalogDBInitializer.cs Remove restock properties from CatalogDBInitializer            eShopOnWeb/eShop.MVC/Models/Infrastructure/CatalogDBInitializer.cs Removed handling of RestockThreshold and MaxStockThreshold in CSV processing. +2/-36    CatalogTable.cshtml Remove restock fields from CatalogTable view                          eShopOnWeb/eShop.MVC/Views/Catalog/CatalogTable.cshtml Removed display fields for RestockThreshold and MaxStockThreshold. +1/-13    Create.cshtml Remove restock fields from Create view                                      eShopOnWeb/eShop.MVC/Views/Catalog/Create.cshtml Removed form fields for RestockThreshold and MaxStockThreshold. +0/-14    Details.cshtml Remove restock fields from Details view                                    eShopOnWeb/eShop.MVC/Views/Catalog/Details.cshtml Removed display fields for RestockThreshold and MaxStockThreshold. +0/-16    Edit.cshtml Remove restock fields from Edit view                                          eShopOnWeb/eShop.MVC/Views/Catalog/Edit.cshtml Removed form fields for RestockThreshold and MaxStockThreshold. +0/-14
Formatting	2 files CatalogService.cs Format CatalogService file                                                              eShopOnWeb/eShop.MVC/Services/CatalogService.cs Removed unnecessary newline at the end of the file. +1/-1      CatalogServiceMock.cs Format CatalogServiceMock file                                                      eShopOnWeb/eShop.MVC/Services/CatalogServiceMock.cs Removed unnecessary newline at the end of the file. +1/-1

---

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

[qodo-code-review]

PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here.

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅ 5 - Fully compliant Compliant requirements: All RestockThreshold and MaxStockThreshold properties and references have been removed from all specified files

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Code Inconsistency The code contains a reference to MaxStockThreshold in line 58 which should have been removed as part of this cleanup Code Smell Unnecessary semicolon after return statement in ComposeCatalogItems method

[qodo-code-review]

PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here.

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
Possible issue	Remove display of deleted property to prevent runtime errors Remove the reference to MaxStockThreshold in the table row since this property has been removed from the model. eShopOnWeb/eShop.MVC/Views/Catalog/CatalogTable.cshtml [58] -@Html.DisplayFor(modelItem => item.MaxStockThreshold) +<!-- Remove this line entirely --> Apply this suggestion Suggestion importance[1-10]: 9 Why: This is a critical fix as displaying a removed property will cause runtime errors. The code needs to be removed to maintain consistency with the model changes.	9
General	Remove redundant semicolon after return statement to improve code clarity Remove the redundant semicolon at the end of the return statement, as it creates unnecessary noise in the code. eShopOnWeb/eShop.MVC/Services/CatalogServiceMock.cs [80-81] return items; -; Apply this suggestion Suggestion importance[1-10]: 3 Why: The redundant semicolon after the return statement is a minor code style issue that affects readability but has no functional impact.	3

💡 Need additional feedback ? start a PR chat