Skip to content

[RTM] Fix warnings in the picker widget.#1256

Merged
discordier merged 2 commits intohotfix/2.0.7from
hotfix/picker-widget
Dec 17, 2018
Merged

[RTM] Fix warnings in the picker widget.#1256
discordier merged 2 commits intohotfix/2.0.7from
hotfix/picker-widget

Conversation

@dmolineus
Copy link
Copy Markdown
Contributor

Description

The picker widget show warnings about missing constants TL_ASSETS_URL and TL_FILES_URL. This PR fixes it by calling Controller::setStaticUrls() which defines the constants.

Checklist

  • Read and understood the CONTRIBUTING guidelines
  • Created tests, if possible
  • All tests passing
  • Extended the README / documentation, if necessary
  • Added myself to the @authors in touched PHP files
  • Checked the changes with phpcq and introduced no new issues

@zonky2 zonky2 changed the title Fix warnings in the picker widget. [RTM] Fix warnings in the picker widget. Dec 17, 2018
@discordier discordier changed the base branch from master to hotfix/2.0.7 December 17, 2018 18:31
Copy link
Copy Markdown
Member

@discordier discordier left a comment

Choose a reason for hiding this comment

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

LGTM

@zonky2 zonky2 added this to the 2.0.7 milestone Dec 17, 2018
@zonky2 zonky2 added the bug A bug! A bug! Fast, squish it! label Dec 17, 2018
@discordier discordier merged commit cb057cf into hotfix/2.0.7 Dec 17, 2018
discordier added a commit that referenced this pull request Dec 17, 2018
This calls `Controller::setStaticUrls()` for pickers to have the constants
available.
@discordier discordier deleted the hotfix/picker-widget branch December 17, 2018 21:48
discordier added a commit that referenced this pull request Dec 19, 2018
Maintenance release.

Fixes:
- #1256 - Fix warning of undefined constants TL_ASSETS_URL and TL_FILES_URL.
- #1257 - Hide sorting hint in popup widgets.
- #1258, #1234 Frontend filter widgets for simple lookup can now be enabled
  and disabled via a checkbox.
- #1259 - Changed the default behavior of the `getFilterOptions` to `usedOnly`
  if no id list is given.
discordier added a commit that referenced this pull request Dec 19, 2018
Maintenance release.

Fixes:
- #1256 - Fix warning of undefined constants TL_ASSETS_URL and TL_FILES_URL.
- #1257 - Hide sorting hint in popup widgets.
- #1258, #1234 Frontend filter widgets for simple lookup can now be enabled
  and disabled via a checkbox.
- #1259 - Changed the default behavior of the `getFilterOptions` to `usedOnly`
  if no id list is given.
discordier added a commit that referenced this pull request Dec 19, 2018
Maintenance release.

Fixes:
- #1256 - Fix warning of undefined constants TL_ASSETS_URL and
  TL_FILES_URL.
- #1257 - Hide sorting hint in popup widgets.
- #1258, #1234 Frontend filter widgets for simple lookup can now be
  enabled and disabled via a checkbox.
- #1259 - Changed the default behavior of the `getFilterOptions` to
  `usedOnly` if no id list is given.

Conflicts:
  src/Attribute/BaseSimple.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug A bug! A bug! Fast, squish it!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants