Sweep: Develop a web app for logistics company for customers to calculate price, checkout, track consignments, and admin panel using codeigniter3

Kayusme commented 10 months ago

Checklist

- [X] `application/controllers/PackageController.php` > • Create a new class PackageController that extends CI_Controller. > • Add a function createPackage that takes the weight, length, width, height, and location variables as parameters and creates a new package in the database. > • Add a function getPackage that takes a package ID as a parameter and retrieves the package details from the database. > • Add a function updatePackage that takes a package ID and the new weight, length, width, height, and location variables as parameters and updates the package in the database. > • Add a function deletePackage that takes a package ID as a parameter and deletes the package from the database. - [X] `application/controllers/PriceCalculator.php` > • Modify the calculatePrice function to take an array of packages as a parameter instead of individual weight, length, width, and height parameters. > • Loop through the array of packages and calculate the price for each package using the calculateVolumetricWeight function and the standard price retrieved from the PriceModel. > • Add the location variables 'From' and 'To' as parameters to the calculatePrice function and use them in the price calculation. - [X] `application/views/price_calculator_view.php` > • Add input fields for the location variables 'From' and 'To' to the form. > • Modify the form to allow the user to add multiple packages. - [X] `application/controllers/AdminController.php` > • Modify the updateStandardPrice function to also update the standard price in the PriceCalculator class. - [X] `application/models/PriceModel.php` > • Add a getStandardPrice function that retrieves the standard price from the database.

sweep-ai[bot] commented 10 months ago

Here's the PR! https://github.com/Kayusme/logistics3/pull/5.

⚡ Sweep Free Trial: I used GPT-3.5 to create this ticket. You have 4 GPT-4 tickets left for the month and 0 for the day. For more GPT-4 tickets, visit our payment portal. To retrigger Sweep, edit the issue.

Install Sweep Configs: Pull Request

Step 1: 🔍 Code Search

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.

https://github.com/Kayusme/logistics3/blob/10e670b882f71769c57d4d6228227274c7129015/README.md#L1-L1 https://github.com/Kayusme/logistics3/blob/10e670b882f71769c57d4d6228227274c7129015/application/controllers/PriceCalculator.php#L1-L21 https://github.com/Kayusme/logistics3/blob/10e670b882f71769c57d4d6228227274c7129015/application/controllers/AdminController.php#L1-L12 https://github.com/Kayusme/logistics3/blob/10e670b882f71769c57d4d6228227274c7129015/application/views/price_calculator_view.php#L1-L43 https://github.com/Kayusme/logistics3/blob/10e670b882f71769c57d4d6228227274c7129015/application/views/admin_view.php#L1-L24

Step 2: 🧐 Snippet Analysis

From looking through the relevant snippets, I decided to make the following modifications:

File Path	Proposed Changes
`application/controllers/PackageController.php`	Create application/controllers/PackageController.php with contents: • Create a new class PackageController that extends CI_Controller. • Add a function createPackage that takes the weight, length, width, height, and location variables as parameters and creates a new package in the database. • Add a function getPackage that takes a package ID as a parameter and retrieves the package details from the database. • Add a function updatePackage that takes a package ID and the new weight, length, width, height, and location variables as parameters and updates the package in the database. • Add a function deletePackage that takes a package ID as a parameter and deletes the package from the database.
`application/controllers/PriceCalculator.php`	Modify application/controllers/PriceCalculator.php with contents: • Modify the calculatePrice function to take an array of packages as a parameter instead of individual weight, length, width, and height parameters. • Loop through the array of packages and calculate the price for each package using the calculateVolumetricWeight function and the standard price retrieved from the PriceModel. • Add the location variables 'From' and 'To' as parameters to the calculatePrice function and use them in the price calculation.
`application/views/price_calculator_view.php`	Modify application/views/price_calculator_view.php with contents: • Add input fields for the location variables 'From' and 'To' to the form. • Modify the form to allow the user to add multiple packages.
`application/controllers/AdminController.php`	Modify application/controllers/AdminController.php with contents: • Modify the updateStandardPrice function to also update the standard price in the PriceCalculator class.
`application/models/PriceModel.php`	Create application/models/PriceModel.php with contents: • Add a getStandardPrice function that retrieves the standard price from the database.

Step 3: 📝 Planning

I have created a plan for writing the pull request. I am now working my plan and coding the required changes to address this issue. Here is the planned pull request:

Implement multiple packages, location variables, and standard price update sweep/multiple-packages

Description

This PR implements the following changes:

Created a new PackageController class to handle the creation, retrieval, update, and deletion of packages.

Modified the PriceCalculator class to handle multiple packages, calculate the price based on volumetric weight or actual weight, and consider the location variables 'From' and 'To'.

Updated the price_calculator_view file to allow users to add multiple packages and input the location variables.

Modified the AdminController class to update the standard price in the PriceCalculator class.

Added a getStandardPrice function to the PriceModel class to retrieve the standard price from the database.

Summary of Changes

Created PackageController class with functions for package management.

Modified PriceCalculator class to handle multiple packages and location variables.

Updated price_calculator_view file to allow multiple packages and input location variables.

Modified AdminController class to update standard price in PriceCalculator class.

Added getStandardPrice function to PriceModel class.

Please review and merge these changes.

Step 4: ⌨️ Coding

File	Instructions	Progress
`application/controllers/PackageController.php`	Create application/controllers/PackageController.php with contents: • Create a new class PackageController that extends CI_Controller. • Add a function createPackage that takes the weight, length, width, height, and location variables as parameters and creates a new package in the database. • Add a function getPackage that takes a package ID as a parameter and retrieves the package details from the database. • Add a function updatePackage that takes a package ID and the new weight, length, width, height, and location variables as parameters and updates the package in the database. • Add a function deletePackage that takes a package ID as a parameter and deletes the package from the database.	✅ Commit `10e670b`
`application/controllers/PriceCalculator.php`	Modify application/controllers/PriceCalculator.php with contents: • Modify the calculatePrice function to take an array of packages as a parameter instead of individual weight, length, width, and height parameters. • Loop through the array of packages and calculate the price for each package using the calculateVolumetricWeight function and the standard price retrieved from the PriceModel. • Add the location variables 'From' and 'To' as parameters to the calculatePrice function and use them in the price calculation.	✅ Commit `f7874a8`
`application/views/price_calculator_view.php`	Modify application/views/price_calculator_view.php with contents: • Add input fields for the location variables 'From' and 'To' to the form. • Modify the form to allow the user to add multiple packages.	✅ Commit `b8c8d83`
`application/controllers/AdminController.php`	Modify application/controllers/AdminController.php with contents: • Modify the updateStandardPrice function to also update the standard price in the PriceCalculator class.	✅ Commit `b8c8d83`
`application/models/PriceModel.php`	Create application/models/PriceModel.php with contents: • Add a getStandardPrice function that retrieves the standard price from the database.	✅ Commit `cb9919d`	I have finished coding the issue. I am now reviewing it for completeness.

Step 5: 🔁 Code Review

Here are my self-reviews of my changes at sweep/multiple-packages.

Here is the 1st review

Thank you for your contribution. Here are some changes that need to be made:

In application/controllers/AdminController.php, please ensure that the PriceCalculator library has a method called updateStandardPrice.

In application/controllers/PackageController.php, the methods createPackage, getPackage, updatePackage, and deletePackage are currently empty. Please implement these methods.

In application/controllers/PriceCalculator.php, please ensure that the PriceModel has a method called getStandardPrice.

In application/models/PriceModel.php, the method getStandardPrice is currently empty. Please implement this method.

Once these changes are made, the code should work as expected.

I finished incorporating these changes.

🎉 Latest improvements to Sweep:

Use Sweep Map to break large issues into smaller sub-issues, perfect for large tasks like "Sweep (map): migrate from React class components to function components"
Getting Sweep to format before committing! Check out Sweep Sandbox Configs to set it up.
We released a demo of our chunker, where you can find the corresponding blog and code.

💡 To recreate the pull request edit the issue title or description. ^{Join Our Discord}

Kayusme commented 10 months ago

Sweep(slow): modify the code to consider volumetric weight. If the volumetric weight is higher, it should be used for price calculation, else use actual weight. Users should be able to add multiple packages at the same time. Introduce the location variables From and To. Admin should be able to update the standard price used for calculation

Kayusme commented 10 months ago

Sweep: modify the code to consider volumetric weight. If the volumetric weight is higher, it should be used for price calculation, else use actual weight. Users should be able to add multiple packages at the same time. Introduce the location variables From and To. Admin should be able to update the standard price used for calculation

Kayusme commented 10 months ago

Sweep: revisit task 2 and debug application/controllers/PackageController.php. implement all the specified tasks and update all other files that may be affected

Kayusme commented 10 months ago

Sweep: redesign the UI using Bootstrap5 CoreUI

Kayusme / logistics3