Writer field - Use regular links instead of permalinks

Kirby 4
1

Hello,

In a writer field, when creating a link to a internal page, as href value in the template is it possible to display the regular page link instead of the permalink ?

I’m trying Kirby 4 and it’s great. The new link dialog makes it possible to link any word(s) to a page, but the href attribute contains a permalink, wich is problematic in my case.
Otherwise, in the context of a blocks field, I don’t see any other option than to use the markdown field, wich is way less convenient for the user. But maybe you see one ?

Thanks for your help.

2

For me linking from internal pages does not work.
After assignment source code looks like this:
<a>notes</a>
The href attribute is missing.
If I call the page in the panel again, the link is not recognized at all.
Is this a (known) bug?
I use the starterkit with 4.0.0-beta.1

page-link
page-link1324×1366 74.6 KB

3

I don’t think so, works for me. Where are you using this, in a text block?

4

External links work. Internal page links do not work.
I have tested a block field in the Starterkit.

block-field
block-field2262×1742 483 KB

5

As I wrote above, I cannot reproduce this in a fresh K4 beta 1 Starterkit, but if you want to report a bug, please do so in the GitHub kirby repo.

6

I don’t see this bug neither.

@texnixe any idea concerning my question ?

7

Hm, I don’t think so, looking at the source code, Kirby recognizes links via the file:// or page:// patterns. So it wouldn’t work with this disabled.

What you can do is disable the pages option and use a custom option, i.e. just insert /notes.

8

Is it possible to hook the render so that it gets the regular link from the permalink ?

9

A hook that replaces the link and actually stores the result in the content wouldn’t be useful (because of what I wrote above), what you can do is replace those links when rendering, via a custom field method.

10

Ok thank you :slight_smile:

I’ll try this approach.

12

It works !

Here is the method :

Kirby::plugin('adrienpayet/process-links', [
  'fieldMethods' => [
      'processLinks' => function ($field) {
          $dom = new DOMDocument();
          @$dom->loadHTML(mb_convert_encoding($field->value(), 'HTML-ENTITIES', 'UTF-8'), LIBXML_HTML_NOIMPLIED | LIBXML_HTML_NODEFDTD);
          $anchorTags = $dom->getElementsByTagName('a');
          $site = site();

          foreach ($anchorTags as $anchorTag) {
              $href = $anchorTag->getAttribute('href');
              $getIdPattern = '/@\/page\/(.*)/';
              $regexResult = preg_match($getIdPattern, $href, $matches);
              $isInternalPageLink = $regexResult && isset($matches[1]);
              $targetId = $matches[1];

              if ($isInternalPageLink) {
                try {
                  $targetPage = $site->find('page://' . $targetId);
                  if (!$targetPage) {
                      throw new Exception("No internal page found.");
                  }
                  $anchorTag->removeAttribute('target');
                  $attributes = [
                      'href' => $targetPage->url(),
                      'class' => 'internal-link',
                      'data-size' => formatSize($targetPage->size()->toInt())
                  ];
                  foreach ($attributes as $key => $value) {
                      $anchorTag->setAttribute($key, $value);
                  }
                } catch (Exception $e) {
                    error_log($e->getMessage());
                }
              }
          }
          return $dom->saveHTML();
      }
  ]
]);
1 Like
13

Hi @Adrieng have you published this plugin somewhere?
I’m trying to do the same thing but your code does not include the functions used :wink:

14

Hi @JuG,

No it’s not published.
Which functions are you talking about ? formatSize() was specific to my project but if you think it could be usefulf to you of course I can share it.

15

hum :thinking: the code is not working in my install but not for the reason I thought. Seems not to be related to the function afterall. I’ll investigate.