Skip to content

Update PageSnippet to respect pretty url if site is using#3

Open
BlackbitDevs wants to merge 571 commits into
11.xfrom
pretty-url-for-site
Open

Update PageSnippet to respect pretty url if site is using#3
BlackbitDevs wants to merge 571 commits into
11.xfrom
pretty-url-for-site

Conversation

@BlackbitDevs

Copy link
Copy Markdown
Owner

This PR allows to use pretty url if document belongs to site.
Now pimcore doesn't use document pretty url if document belongs to site https://github.com/pimcore/pimcore/blob/11.x/models/Document/PageSnippet.php#L570

bluvulture and others added 30 commits September 13, 2024 10:06
* Check latitude and longitude for mandatory check

* $isEmpty check costs less
Revert "Revert "Extended example usage (pimcore#17587)""

This reverts commit c1ed3a0.
* added grid preview for field collections

* added grid preview for field collections

* Apply php-cs-fixer changes
Added ignore list for vulnerabilities check
…alue (pimcore#17638)

* filter_priority uses null as default values

* filter_priority revert back to scalar node
In version 11.3.1 migration Version20230412105530.php was removed and replaced by Version20240708083500. Yet the documentation was not updated to reflect this change. Which causes confusion when upgrading to Pimcore 11
…lity with twig/twig 3.14 (pimcore#17593)

* fix: pimcore#17582 test `pimcore_placeholder` with twig 3.14

* Apply php-cs-fixer changes

* fix: use ArrayLoader instead of FilesystemLoader for twig tests

* Apply suggestions from code review

---------

Co-authored-by: michalananapps <michalananapps@users.noreply.github.com>
Co-authored-by: JiaJia Ji <kingjia90@gmail.com>
* [SeoBundle] Add redirect target path resolution

When reading redirects with php code the target can be either a path or a site id.
The type of the redirect does not reflect the nature of the target link.
With type `path` it can either be an id or a path string.
Therefore an additional method for getting the actual target path is required.

* [SeoBundle] Add documentation to destinct methods

* Apply php-cs-fixer changes

---------

Co-authored-by: dweiss96 <dweiss96@users.noreply.github.com>
Update Bugfix branch.
…list

Update composer-checks-vulnerabilities.yaml
kingjia90 and others added 29 commits February 24, 2025 12:26
* refactor with a continue, as the expandGridColumnForExport doesn't work if isHelperGridColumnConfig is false

* Apply php-cs-fixer changes

---------

Co-authored-by: kingjia90 <6014195+kingjia90@users.noreply.github.com>
* Translatable log levels

* Code format

* Removed unnecessary toArray method
* remove deprecated headless chromium

* indirectly remove gotenberg v1 support due to being "tied" to headless chrome

* Update README.md

* Apply suggestions from code review

Co-authored-by: Sebastian Blank <blank@data-factory.net>

* Update GotenbergHelper.php

* fix conflict merge

* Update composer.json

* Update composer.json

---------

Co-authored-by: Sebastian Blank <blank@data-factory.net>
… config values to handle nested arrays (pimcore#18145)

* Refactor config value replacement to handle nested arrays

* Apply php-cs-fixer changes

---------

Co-authored-by: fashxp <8792145+fashxp@users.noreply.github.com>
* Fix PhpDocs

* Apply php-cs-fixer changes

* Fix new phpstan error

---------

Co-authored-by: blankse <998558+blankse@users.noreply.github.com>
* [Task]: Require PHPStan v2.0

* Remove gotenberg-php v1 Support

* Use @phpstan-ignore-next-line for include_once (every system have other paths)

* Fix Error with PHP 8.4 (use is_subclass_of instead of ReflectionClass::isSubclassOf())

* Fix Sonar bugs

* Fix Sonar bugs

* Update lib/Security/User/ObjectUserProvider.php

Co-authored-by: Jacob Dreesen <jacob@hdreesen.de>

* Review

* Update auf phpstan 2.1.3

* Update phpstan to version 2.1.4

* Fix new phpstan error

* Update phpstan to version 2.1.5

* Generate baseline

* Update auf phpstan 2.1.6, Fix new errors

* Fix prepareDataForPersistence() methods

* Update ergebnis/phpstan-rules to fix lowest tests

---------

Co-authored-by: Jacob Dreesen <jacob@hdreesen.de>
…ms other than password_hash (pimcore#18156)

* Removed deprecated pw hash methods and algorithms

* Reverted code change for debugging

* Removed method accessing removed properties

* Upgrade notes

* Updated documentation

* Clarified upgrade notes, thanks to AI
…transition label. (pimcore#18096)

* Pass the transition itself into notification service rather than the transition label.

* docs

* typo
pimcore#18163)

* First version of MimeTypeHelper with file and stream support

* Changed fallback and default value

* Code format

* Fix phpstan

* Avoid creating a tmp file for stream

* Switch to non static methods

* Fixed php doc
Co-authored-by: JiaJia Ji <kingjia90@gmail.com>
Co-authored-by: JiaJia Ji <kingjia90@gmail.com>
kingjia90 pushed a commit that referenced this pull request Aug 29, 2025
Can not filter checkbox values in Grid.

Fatal Error
File - vendor/doctrine/dbal/src/Connection.php
Line - 568
Trace - Doctrine\DBAL\Connection::quote(): Argument #1 ($value) must be of type string, int given, called in vendor/pimcore/pimcore/models/DataObject/ClassDefinition/Data/Checkbox.php on line 174
Trace - #0 vendor/pimcore/pimcore/models/DataObject/ClassDefinition/Data/Checkbox.php(174): Doctrine\DBAL\Connection->quote()
#1 vendor/pimcore/pimcore/models/DataObject/ClassDefinition/Data/Checkbox.php(158): Pimcore\Model\DataObject\ClassDefinition\Data\Checkbox->getFilterConditionExt()
#2 vendor/pimcore/admin-ui-classic-bundle/src/Helper/GridHelperService.php(366): Pimcore\Model\DataObject\ClassDefinition\Data\Checkbox->getFilterCondition()
#3 vendor/pimcore/admin-ui-classic-bundle/src/Helper/GridHelperService.php(667): Pimcore\Bundle\AdminBundle\Helper\GridHelperService->getFilterCondition()
pimcore#4 vendor/pimcore/admin-ui-classic-bundle/src/Controller/Admin/DataObject/DataObjectActionsTrait.php(102): Pimcore\Bundle\AdminBundle\Helper\GridHelperService->prepareListingForGrid()
pimcore#5 vendor/pimcore/admin-ui-classic-bundle/src/Controller/Admin/DataObject/DataObjectController.php(1693): Pimcore\Bundle\AdminBundle\Controller\Admin\DataObject\DataObjectController->gridProxy()
pimcore#6 vendor/symfony/http-kernel/HttpKernel.php(181): Pimcore\Bundle\AdminBundle\Controller\Admin\DataObject\DataObjectController->gridProxyAction()
pimcore#7 vendor/symfony/http-kernel/HttpKernel.php(76): Symfony\Component\HttpKernel\HttpKernel->handleRaw()
pimcore#8 vendor/symfony/http-kernel/Kernel.php(197): Symfony\Component\HttpKernel\HttpKernel->handle()
pimcore#9 vendor/symfony/runtime/Runner/Symfony/HttpKernelRunner.php(35): Symfony\Component\HttpKernel\Kernel->handle()
pimcore#10 vendor/autoload_runtime.php(29): Symfony\Component\Runtime\Runner\Symfony\HttpKernelRunner->run()
pimcore#11 public/index.php(19): require_once('...')
pimcore#12 {main}
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.