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

Upgrader - CLI - custom_avatar & $upgradeData #6001

Closed
sbulen opened this issue Feb 5, 2020 · 2 comments · Fixed by #6036
Closed

Upgrader - CLI - custom_avatar & $upgradeData #6001

sbulen opened this issue Feb 5, 2020 · 2 comments · Fixed by #6036
Labels

Comments

@sbulen
Copy link
Contributor

sbulen commented Feb 5, 2020

Description

Two issues specific to upgrading via the CLI:

  • If the prior version didn't have a custom_avatar directory, it is somehow not created upon upgrade
  • $upgradeData is left in Settings.php when done

Steps to reproduce

  1. Do an upgrade via CLI on a pure vanilla 2.0 or 1.x implementation with no custom avs directory setup

Environment (complete as necessary)

  • Version/Git revision: Today
  • Database Type: mysql
  • Database Version: 5.7
  • PHP Version: 7.3

Additional information/references

Errors from trying to upload an avatar:
image

The problem is the folder & url weren't setup by the upgrader:
image

$upgradeData is still in Settings.php:
image

@sbulen sbulen added the Upgrader label Feb 5, 2020
@sbulen
Copy link
Contributor Author

sbulen commented Feb 5, 2020

I'm not quite sure how deep to go here. I have been focusing on reporting errors from 2.0=>2.1 upgrades thus far via CLI. 1.x=>2.1 produces many, many more errors & gaps in functionality.

Browser upgrades are far more robust than CLI at the moment... Especially the further back you go.

@jdarwood007
Copy link
Member

Focus mostly on 2.0 to 2.1. Unless you can prove that the 1.1 to 2.0 worked and the 1.1 to 2.1 does not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants