Need to make customizable the GitHub onebox number of lines limitation instead of the current 20

dmitry_fedyuk · Avril 10, 2016, 5:21

For example, the following method contains 22 lines of code, and it is impossible to publish it in a useful way.
It is only possible to publish it as…

chopped:

github.com/magento/magento2

app/code/Magento/Security/Model/ResourceModel/AdminSessionInfo/Collection.php

052e789


      
          /**
           * Update active sessions status except a specific one
           *
           * @param int $status
           * @param int $userId
           * @param string $sessionIdToExclude
           * @param int $updateOlderThen
           * @return int The number of affected rows.
           */
          public function updateActiveSessionsStatus(
              $status,
              $userId,
              $sessionIdToExclude,
              $updateOlderThen = null
          ) {
              return $this->getResource()->updateStatusByUserId(
                  $status,
                  $userId,
                  [\Magento\Security\Model\AdminSessionInfo::LOGGED_IN],
                  [$sessionIdToExclude],

This file has been truncated. show original

or splitted:

github.com/magento/magento2

app/code/Magento/Security/Model/ResourceModel/AdminSessionInfo/Collection.php

052e789


      
          /**
           * Update active sessions status except a specific one
           *
           * @param int $status
           * @param int $userId
           * @param string $sessionIdToExclude
           * @param int $updateOlderThen
           * @return int The number of affected rows.
           */
          public function updateActiveSessionsStatus(
              $status,
              $userId,
              $sessionIdToExclude,
              $updateOlderThen = null
          ) {

github.com/magento/magento2

app/code/Magento/Security/Model/ResourceModel/AdminSessionInfo/Collection.php

052e789


      
              return $this->getResource()->updateStatusByUserId(
                  $status,
                  $userId,
                  [\Magento\Security\Model\AdminSessionInfo::LOGGED_IN],
                  [$sessionIdToExclude],
                  $updateOlderThen
              );
          }

It would be nice to let the forum’s administrator to decide what should be the number of lines limitation instead of the currently 20.

riking · Avril 10, 2016, 8:24

Meh, if you need to share that much code copy it into a backtick-fenced block and link to the source on the next line (with a onebox breaker, such as “Source: link”).

dmitry_fedyuk · Octobre 6, 2016, 4:07

I have implemented it as a plugin: GitHub - discourse-pro/df-github-blob-onebox: The plugin removes the limitation on the maximum number of lines of code of the default GitHub blob onebox for the Discourse forum software
A usage example: How does \Magento\Sales\Model\Order\CustomerManagement::create() work? - Magento 2

Sujet		Réponses	Vues
Line context for GitHub links in onebox Support	5	623	Juillet 2, 2020
Github boxes should be truncated Feature	7	1569	Janvier 27, 2015
Option to not truncate GitHub/GitLab onebox Feature	5	727	Septembre 3, 2018
Pastebin onebox escapes page width Bug	3	1286	Février 22, 2016
Feature the related anchor in github URLs with only one line? Feature	8	1791	Décembre 4, 2025

Need to make customizable the GitHub onebox number of lines limitation instead of the current 20

Sujets connexes