removed restock everywhere

[mw3155]

PR Type

enhancement

---

Description

• Removed all references to restockthreshold from the CatalogDBInitializer, including CSV processing and item creation logic.
• Updated the Create, Delete, and Edit views to remove the RestockThreshold field, simplifying the user interface.
• Minor formatting adjustments in the CatalogDBInitializer file.

---

Changes walkthrough 📝

Relevant files

Enhancement

CatalogDBInitializer.cs Remove `restockthreshold` handling from CatalogDBInitializer eShopOnWeb/eShop.MVC/Models/Infrastructure/CatalogDBInitializer.cs Removed handling of restockthreshold from CSV processing. Deleted code related to restockthreshold in CreateCatalogItem. Minor formatting changes. +4/-21    Create.cshtml Remove `RestockThreshold` input from Create view                  eShopOnWeb/eShop.MVC/Views/Catalog/Create.cshtml Removed RestockThreshold form group from the Create view. +0/-7      Delete.cshtml Remove `RestockThreshold` display from Delete view              eShopOnWeb/eShop.MVC/Views/Catalog/Delete.cshtml Removed RestockThreshold display from the Delete view. +0/-8      Edit.cshtml Remove `RestockThreshold` input from Edit view                      eShopOnWeb/eShop.MVC/Views/Catalog/Edit.cshtml Removed RestockThreshold form group from the Edit view. +0/-7

---

💡 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:

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

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Error Handling The error message for header count validation has been reformatted but the throw statement was accidentally removed, which could lead to silent failures when processing invalid CSV files Data Validation Removal of restock threshold validation may require additional checks or updates to ensure inventory management logic remains valid without this field

[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	Restore missing validation check for required CSV headers The if condition for checking required headers is empty, which silently ignores the validation. Re-add the throw statement to prevent data corruption. eShopOnWeb/eShop.MVC/Models/Infrastructure/CatalogDBInitializer.cs [283-286] if (csvheaders.Count() < requiredHeaders.Count()) { + throw new Exception($"requiredHeader count '{requiredHeaders.Count()}' is bigger then csv header count '{csvheaders.Count()}' "); } Apply this suggestion Suggestion importance[1-10]: 9 Why: The suggestion fixes a critical data validation issue where the empty if block would silently ignore missing required headers, potentially leading to data corruption. The fix restores the necessary exception throwing mechanism.	9
	Add missing error handling for invalid integer parsing The exception message for invalid maxStockThreshold is missing in the code. Add proper error handling similar to other validation checks. eShopOnWeb/eShop.MVC/Models/Infrastructure/CatalogDBInitializer.cs [245] string maxStockThresholdString = column[maxStockThresholdIndex].Trim('"').Trim(); +if (!String.IsNullOrEmpty(maxStockThresholdString)) +{ + if (int.TryParse(maxStockThresholdString, out int maxStockThreshold)) + { + catalogItem.MaxStockThreshold = maxStockThreshold; + } + else + { + throw new Exception($"maxStockThreshold={maxStockThresholdString} is not a valid integer"); + } +} Apply this suggestion Suggestion importance[1-10]: 8 Why: The suggestion addresses a critical issue where missing error handling could lead to silent failures or data corruption when parsing maxStockThreshold values. The improved code adds proper validation and error reporting consistent with other similar checks in the codebase.	8

💡 Need additional feedback ? start a PR chat