Twingly Pane is an easy to install and configure pane to use with ctools and page manager to show a Twingly widget on your site for certain URLs. Twingly is a blog and microblog search.

Requirements

Usage

Install and activate.
In Page manager you could add a Twingly pane from the category Simple pane widgets. You can now add a URL to see Twingly links for.

Sandbox

Project page: http://drupal.org/sandbox/mikkeX/1363048

Repository

git clone --branch 7.x-1.x http://git.drupal.org/sandbox/mikkeX/1363048.git

Comments

misc’s picture

Issue summary: View changes

Better explained...

patrickd’s picture

Status: Needs review » Needs work

You got a few formatting errors: http://ventral.org/pareview/httpgitdrupalorgsandboxmikkex1363048git
It seems that you already tested it your self, do you have any questions on that?

misc’s picture

Thanks. Yeah I tested it, the errors that are still left I do not understand how to fix.

patrickd’s picture

  • 27 | ERROR | Missing parameter type at position 1
     * @param $conf
     *   Configuraton that are stored.

    ->

     * @param array $conf
     *   Configuraton that are stored.
  • 33 | ERROR | Data type of return value is missing
     * @return
     *   Render the pane

    ->

     * @return object
     *   Render the pane
  • 46 | ERROR | Doc comment for var The does not match actual variable name
    | | &$form at position 1

    46 | ERROR | Parameter comment must be on the next line at position 1

     * @param &$form
     *   The form that are used.
     *
     * @param &$form_state
     *   States of the form settings.
     *
     */
    function twingly_pane_pane_content_type_edit_form(&$form, &$form_state) {
    

    ->

     * @param $form
     *   The form that are used.
     * @param $form_state
     *   States of the form settings.
     *
     */
    function twingly_pane_pane_content_type_edit_form(&$form, &$form_state) {
  • 89 | ERROR | String concat is not required here; use a single string instead
    $output = '<div class="twingly_widget">'
                . '<a href="http://www.twingly.com">' . t('Twingly Blog Search') . '</a>'
                . ' <span class="query">link:' . filter_xss($url_twingly_pings) . ' sort:published</span>'
                . '</div>';
    $output = '<div class="twingly_widget"><a href="http://www.twingly.com">' 
               . t('Twingly Blog Search') . '</a> <span class="query">link:'
               . filter_xss($url_twingly_pings) . ' sort:published</span></div>';
  • 4 | ERROR | The second line in the file doc comment must be " * @file"
    Put a @file comment block at the beginning of twingly_pane.module and describe the function of the file just like in twingly_pane_pane.inc

Hope this clarifies some issues!

misc’s picture

Status: Needs work » Needs review

Thanks for the explanation!
Have now updated the code and it seems there are no more formatting errors.

misc’s picture

Just a friendly reminder if somebody would like to test this module, if you have ctools installed it is really easy to test.

patrickd’s picture

Sorry it seems like you got two application issues opened (http://drupal.org/node/1336946)
Unfortunatelly you can only hold one of these open, please close one of them!

(I'd suggest you to close the more recent one)

misc’s picture

Status: Needs review » Closed (duplicate)

Closed because of having two applications in the cue.

misc’s picture

Issue summary: View changes

spelling

avpaderno’s picture

Title: Twingly Pane » [D7] Twingly Pane
Issue summary: View changes
Related issues: +#1336946: [D7] Entity list display