Skip to content

[ticket/12524] Add search.php core event to modify search results rowset#2437

Merged
nickvergessen merged 1 commit intophpbb:develop-ascraeusfrom
rxu:ticket/12524
May 11, 2014
Merged

[ticket/12524] Add search.php core event to modify search results rowset#2437
nickvergessen merged 1 commit intophpbb:develop-ascraeusfrom
rxu:ticket/12524

Conversation

@rxu
Copy link
Copy Markdown
Contributor

@rxu rxu commented May 11, 2014

Currently it is possible to change search results within the search row scope,
but it should be possible to modify/unset every individual row
within the rowset.

PHPBB3-12524.

@nickvergessen
Copy link
Copy Markdown
Contributor

phpbb_event_export_php_test::test_crawl_php_file with data set #1 ('search.php')

LogicException: Should use multiple lines for $vars definition for event 'core.search_modify_tpl_ary' in file 'search.php:927'

phpBB/search.php Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

    $vars = array(
        'attachments',
        'hilit',
        ....
    );

@rxu
Copy link
Copy Markdown
Contributor Author

rxu commented May 11, 2014

Updated #2437.

Currently it is possible to change search results within the search row scope,
but it should be possible to modify/unset every individual row
within the rowset.

PHPBB3-12524
nickvergessen added a commit to nickvergessen/phpbb that referenced this pull request May 11, 2014
[ticket/12524] Add search.php core event to modify search results rowset

* rxu/ticket/12524:
  [ticket/12524] Add search.php core event to modify search results rowset
@nickvergessen nickvergessen merged commit 1b24d68 into phpbb:develop-ascraeus May 11, 2014
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