Skip to content
Snippets Groups Projects

Feature/attributes

Merged Krzysztof Dyszczyk requested to merge feature/attributes into master
1 unresolved thread

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Krzysztof DyszczykKrzysztof Dyszczyk 6 years ago (Oct 12, 2018 7:32pm UTC)

Merge details

  • Changes merged into master with 55405eb4.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
64 64 * @param string $noticeType Notice type.
65 65 * @param bool $dismissible Is dismissible.
66 66 * @param int $priority Notice priority.
67 */
68 public function __construct($noticeContent, $noticeType = 'info', $dismissible = false, $priority = 10)
69 {
67 * @param array $attributes Attributes.
68 */
69 public function __construct(
70 $noticeContent,
71 $noticeType = 'info',
72 $dismissible = false,
  • btw. najczęściej(nie zawsze) taki parametr bool oznacza, że powinny istnieć klasy, jedna która ma zawsze dissmisible = true i druga która ma zawsze dissmisible = false. Wtedy parametr dissmisible nie jest w ogóle potrzebny bo ta informacja jest zawarta w istocie tego czym są te nowe klasy. np. DissmisibleNotice, NoDismissibleNotice :)

    wtedy PermanentDismissibleNotice extends DissmisibleNotice :)

    By Dyszczo on 2018-10-12T19:32:47 (imported from GitLab)

  • Please register or sign in to reply
  • merged

    By Dyszczo on 2018-10-12T19:32:53 (imported from GitLab)

  • mentioned in commit 55405eb4

    By Dyszczo on 2018-10-12T19:32:53 (imported from GitLab)

  • Please register or sign in to reply
    Loading