Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Development backup #51

Merged
merged 52 commits into from
Jan 9, 2025
Merged

Conversation

AndresDiazCarrillo
Copy link
Collaborator

No description provided.

germanp24 and others added 30 commits November 27, 2024 18:06

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
update admin_page

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Download all new changes
…age and the controller for filtering.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Gen restaurants

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
refrech gen_menu

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Addition of the search engine by functional location to the homepage

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Merge pull request #38 from germanp24/gen_homePage

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Branch update

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Gen restaurants

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
 Update branch

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Gen home page
germanp24 and others added 22 commits January 5, 2025 17:49
…erDetails” page should be removed because we have merged it with “menuDetails”.

We had to add a new entity to manage the cart.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Update of all gen_menu changes
…veryManController, and HomeClientController
…ontroller, IndexController, and MenuItemController
…date access modifiers, and add new tests for Address, CartItem, MenuContent, and Usuary.
…and redundant test methods, update access modifiers in MenuItemTest
@AndresDiazCarrillo AndresDiazCarrillo merged commit d928a31 into gen_restaurants Jan 9, 2025
3 checks passed
Copy link

sonarqubecloud bot commented Jan 9, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
52.3% Coverage on New Code (required ≥ 80%)
B Security Rating on New Code (required ≥ A)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

model.addAttribute("dni", dni); // Pasar el DNI al modelo
model.addAttribute("method", method); // Pasar el método de pago al modelo
log.info("DNI: {}, Método de pago: {}", dni, method);

Check notice

Code scanning / SonarCloud

Logging should not be vulnerable to injection attacks Low

Change this code to not log user-controlled data. See more on SonarQube Cloud
System.out.println("Dirección recibida: " + address);


log.info("Dirección recibida para DNI {}: {}", dni, address);

Check notice

Code scanning / SonarCloud

Logging should not be vulnerable to injection attacks Low

Change this code to not log user-controlled data. See more on SonarQube Cloud
// Aquí solo imprimimos los datos como ejemplo
System.out.println("Dirección recibida: " + address);
System.out.println("Tarjeta de crédito recibida: " + creditCard);
log.info("Dirección recibida para DNI {}: {}", dni, address);

Check notice

Code scanning / SonarCloud

Logging should not be vulnerable to injection attacks Low

Change this code to not log user-controlled data. See more on SonarQube Cloud
System.out.println("Dirección recibida: " + address);
System.out.println("Tarjeta de crédito recibida: " + creditCard);
log.info("Dirección recibida para DNI {}: {}", dni, address);
log.info("Tarjeta de crédito recibida: {}", creditCard);

Check notice

Code scanning / SonarCloud

Logging should not be vulnerable to injection attacks Low

Change this code to not log user-controlled data. See more on SonarQube Cloud
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants