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

BS5 vorbereitung #962

Closed
wants to merge 2 commits into from
Closed

Conversation

hhunderter
Copy link
Member

Description

Versionskontrolle BS5
Module und Layouts überarbeitet
Vorbereitung für #454

Type of change

  • New feature (non-breaking change which adds functionality)

This PR has been tested in the following browsers:

  • Chrome
  • Firefox
  • Opera
  • Edge

Versionskontrolle BS5
Module und Layouts überarbeitet
Vorbereitung für IlchCMS#454
@hhunderter hhunderter closed this Apr 17, 2024
blackcoder87 added a commit that referenced this pull request Jun 23, 2024
blackcoder87 added a commit that referenced this pull request Aug 7, 2024
- Code style improvements.
- Remove curlOpt array as the stored cURL options where never used after saving them in that array. The way the options where stored wasn't useful either as a cURL option is "tied" to a specific transfer (transferUrl). This shouldn't affect the functionality as the important function call for cURL is curl_setopt().
- Remove getVersionsList() as it was only used in newVersionFound() of the transfer class itself and just one time. A second call was unnecessary.
- Don't set zipFile in newVersionFound(). This caused an error (under PHP 8.2) and looks like it wasn't needed.
- Verify file right before installing.
- BS5 vorbereitung #962
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.

1 participant