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

Adding old hacks (not included into 1.3.6) into release-1.3 branch #1511

Merged
1 commit merged into from
Dec 27, 2013
Merged

Adding old hacks (not included into 1.3.6) into release-1.3 branch #1511

1 commit merged into from
Dec 27, 2013

Conversation

jguille2
Copy link
Contributor

Adding old PRs and hacks of 1.3 branch, lost in 1.3.6 release.

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #801,#598,#593,#616,#617,#618,#621,#564
Refs tickets
License MIT
Doc PR -

Some details:
#801 (incl #665) -> PR #924 - 'extmenu drag and drop1 -> /system/Admin/javascript/admin_admin_ajax.js
#598 -> 'EntityAccess::toArray - don't call non existing methods' -> lib/Zikula/EntityAccess.php
#593 -> PR #607 ->'Get modimage by ModUtil::getModuleImagePath()' -> lib/util/ModUtil.php, lib/viewplugins/function.modgetimage.php, system/Admin/lib/Admin/Controller/Admin.php, system/Admin/templates/admin_admin_header.tpl

55a22f1 -> 'replaced double quotes with single quotes in gettext string, because it gave E_NOTICE' ->system/Extensions/templates/extensions_hookui_hooks.tpl

PR #616,#617,#618 -> /lib/dbobject/DBObject.php I DBObjectArray.php
#621 -> PR #622 -> /style/core.css
#564 -> /system/Admin/templates/admin_admin_developernotices.tpl

ghost pushed a commit that referenced this pull request Dec 27, 2013
Adding old hacks (not included into 1.3.6) into release-1.3 branch
@ghost ghost merged commit 13305aa into zikula:release-1.3 Dec 27, 2013
@rallek
Copy link
Contributor

rallek commented Jan 5, 2014

can we get this also in the ci.zikula.org?

This pull request was closed.
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.

2 participants