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

Implemented EZP-20787: fiximagesoutsidevardir improvements #623

Closed
wants to merge 5 commits into from

Conversation

bdunogier
Copy link
Member

See https://jira.ez.no/browse/EZP-20787.

Makes the update script support the use-case where both the setting AND the vardir were changed, but existing images references haven't been updated.

Bertrand Dunogier added 5 commits April 29, 2013 16:20
If the var folder was renamed after the setting was changed, the database
contains references to imates in the old path.

The update script is now able to detect this, and will update the database
references without trying to move the files.
}
else
{
$moveFiles = true;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/moveFiles/moveFile/ ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. I can actually completely remove this block.

@lolautruche
Copy link
Contributor

Apart remark from @patrickallaert, +1

@patrickallaert
Copy link
Contributor

Other than my remark: +1

@andrerom
Copy link
Contributor

+1 for >=5.1 with fix for issue pointed out by @patrickallaert

@bdunogier
Copy link
Member Author

Alright, enough +1, merging (Patrick's remark fixed in 6326e45).

@lolautruche
Copy link
Contributor

Manually merged in c684e25

@lolautruche lolautruche deleted the impl-EZP-20787-fximagesvardir_improvements branch April 30, 2013 15:50
ernestob added a commit to ernestob/ezpublish-legacy that referenced this pull request Sep 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants