Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/1.2' into 1.2
Browse files Browse the repository at this point in the history
  • Loading branch information
mcop1 committed Sep 24, 2024
2 parents c2f2b49 + 1d00528 commit 92d32ef
Show file tree
Hide file tree
Showing 8 changed files with 17 additions and 21 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/static-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,10 @@ jobs:
- { php-version: "8.3", dependencies: "highest", experimental: false }
steps:
- name: "Checkout code"
uses: "actions/checkout@v2"
uses: actions/checkout@v4

- name: "Install PHP"
uses: "shivammathur/setup-php@v2"
uses: shivammathur/setup-php@v2
with:
coverage: "none"
php-version: "${{ matrix.php-version }}"
Expand All @@ -47,7 +47,7 @@ jobs:
composer require --no-update pimcore/pimcore:"${PIMCORE_VERSION}"
fi
- name: "Install dependencies with Composer"
uses: "ramsey/composer-install@v2"
uses: ramsey/composer-install@v3
with:
dependency-versions: "${{ matrix.dependencies }}"

Expand All @@ -60,7 +60,7 @@ jobs:

- name: "Upload baseline file"
if: ${{ failure() }}
uses: actions/upload-artifact@v2
uses: actions/upload-artifact@v4
with:
name: phpstan-baseline.neon
path: phpstan-baseline.neon
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
#### v1.2.1
- Removed the package "rybakit/twig-deferred-extension". If you extend the twig layout from the E-Commerce Framework, please check if custom CSS/JS code added by `pimcore_head_script` and `pimcore_head_link` is still working.
4 changes: 0 additions & 4 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
"pimcore/google-marketing-bundle": "^1.0",
"knplabs/knp-paginator-bundle": "^6.0.0",
"symfony/webpack-encore-bundle": "^1.13.2",
"rybakit/twig-deferred-extension": "^3.0",
"symfony/form": "^6.2"
},
"require-dev": {
Expand All @@ -33,9 +32,6 @@
"elasticsearch/elasticsearch": "^8.0",
"pimcore/elasticsearch-client": "^1.0.0"
},
"conflict": {
"twig/twig": "^3.9.0"
},
"suggest": {
"elasticsearch/elasticsearch": "Required for Elastic Search service",
"webmozarts/console-parallelization": "Required for parallelization of console commands"
Expand Down
5 changes: 0 additions & 5 deletions src/Resources/config/services.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,3 @@ services:
resource: '../../Controller'
public: true
tags: ['controller.service_arguments']

# the deferred extension is needed for placeholder helpers to work
# as otherwise the placeholder block would be rendered before any
# content was added (e.g. headTitle)
Twig\DeferredExtension\DeferredExtension: ~
5 changes: 3 additions & 2 deletions src/Resources/views/admin_order/detail.html.twig
Original file line number Diff line number Diff line change
Expand Up @@ -411,7 +411,7 @@
</div>
</div>

{% do pimcore_head_script().captureStart() %}
{% set inlineScript %}
document.addEventListener("DOMContentLoaded", function(event) {
// pimcore open object
document.querySelectorAll('[data-action=open]').forEach(function(el){
Expand Down Expand Up @@ -448,5 +448,6 @@
{% endif %}

});
{% do pimcore_head_script().captureEnd() %}
{% endset %}
{% do pimcore_head_script().appendScript(inlineScript ~ "") %}
{% endblock %}
6 changes: 4 additions & 2 deletions src/Resources/views/admin_order/list.html.twig
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@
{% include "@PimcoreEcommerceFramework/includes/paging.html.twig" with {'paginationVariables': paginator.getPaginationData() } %}
{% endif %}

{% do pimcore_head_script().captureStart() %}
{% set inlineScript %}
//datepicker
flatpickr(".date", {allowInput: true});

Expand All @@ -116,7 +116,9 @@
});
});

{% do pimcore_head_script().captureEnd() %}
{% endset %}

{% do pimcore_head_script().appendScript(inlineScript ~ "") %}
{% endblock %}


Expand Down
4 changes: 2 additions & 2 deletions src/Resources/views/back-office.html.twig
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
{% do pimcore_head_script().prependFile(asset('/bundles/pimcoreecommerceframework/vendor/bootstrap4/js/bootstrap-native-v4.min.js')) %}
{% endapply %}

{% block head_stylesheets deferred %}
{% block head_stylesheets %}
{{ pimcore_head_link() }}
{% endblock %}

Expand Down Expand Up @@ -79,7 +79,7 @@
</div>

{# output scripts added before #}
{% block scripts deferred %}
{% block scripts %}
{{ pimcore_head_script() }}
{% endblock %}

Expand Down
4 changes: 2 additions & 2 deletions src/Resources/views/voucher/voucher_layout.html.twig
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,13 @@
{% do pimcore_head_script().appendFile(asset('/bundles/pimcoreecommerceframework/js/voucherservice/voucherSeriesTabScript.js')) %}
{% endapply %}

{% block head_stylesheets deferred %}
{% block head_stylesheets %}
{{ pimcore_head_link() }}
{% endblock %}
</head>
<body>
{# output scripts added before #}
{% block scripts deferred %}
{% block scripts %}
{{ pimcore_head_script() }}
{% endblock %}
{% set colors = {
Expand Down

0 comments on commit 92d32ef

Please sign in to comment.