Skip to content

Commit 7302b5e

Browse files
authored
Removed app/code/core/Zend (#2861)
1 parent a7fdf56 commit 7302b5e

File tree

3 files changed

+6
-1869
lines changed

3 files changed

+6
-1869
lines changed

README.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -178,14 +178,18 @@ If your project uses OpenMage through composer then all dependencies will be man
178178
If you just extracted the release zip/tarball in your project's main folder then be sure to:
179179
- remove the old copy of aforementioned libraries from your project, you can do that with this command:
180180
```bash
181-
rm -rf lib/Cm lib/Credis lib/mcryptcompat lib/Pelago lib/phpseclib lib/Zend
181+
rm -rf app/code/core/Zend lib/Cm lib/Credis lib/mcryptcompat lib/Pelago lib/phpseclib lib/Zend
182182
```
183183

184-
185184
- download the new release zip file that is named `openmage-VERSIONNUMBER.zip`, this one is built to contain the `vendor`
186185
folder generated by composer, with all the dependencies in it
187186
- extract the zip file in your project's repository as you always did
188187

188+
We also decided to remove our Zend_DB patches (that were stored in `app/code/core/Zend`) because they were very old and
189+
not compatible with the new implementations made by ZF1-Future, which is much more advanced and feature rich.
190+
This may generate a problem with `Zend_Db_Select' statements that do not use 'Zend_Db_Expr' to quote expressions.
191+
If you see SQL errors after upgrading please remember to check for this specific issue in your code.
192+
189193
### New Config Options
190194

191195
- `admin/design/use_legacy_theme`

0 commit comments

Comments
 (0)