[Tiki-devel] Gitlab trouble , was: Re: Indentation plugin

Volker Wysk <post-hhF2Jplw28UoZk/[email protected]>
Newsgroups gmane.comp.cms.tiki.devel
Message-ID <[email protected]>
I had messed up my repositories (local/gitlab). I've started from scratch.
Now everything is clean and the merge request is in place.

Volker

Am Sonntag, dem 12.12.2021 um 17:23 +0100 schrieb Volker Wysk:
> The new merge request also doesn't appear on the merge requests page:
> https://gitlab.com/tikiwiki/tiki/-/merge_requests
> 
> 
> Am Sonntag, dem 12.12.2021 um 17:18 +0100 schrieb Volker Wysk:
> > Something's gone wrong. I've made a merge request and got this error message
> > from Gitlab in an email: "Tiki | Failed pipeline for ...". But the message
> > body says: "Pipeline #427803129 triggered by Volker Wysk had 0 failed jobs."
> > 
> > Then I clicked on "Merge" in the Gitlab page, and the problem seems to have
> > gone away.
> > 
> > WTF?
> > 
> > Cheers, Volker
> > 
> > 
> > 
> > Am Sonntag, dem 12.12.2021 um 17:02 +0100 schrieb Volker Wysk:
> > > Am Sonntag, dem 12.12.2021 um 12:27 +0000 schrieb Jonny Bradley via
> > > TikiWiki-devel:
> > > > Hi Volker
> > > > 
> > > > Good stuff, you know you can make a draft merge request on gitlab and we
> > > > could discuss it there, but here's good too!
> > > > 
> > > > Re the undocumented properties:
> > > > 
> > > > >        'introduced' => 24,
> > > > 
> > > > As you guessed, is the Tiki version it first appeared in, so 24 for this
> > > > is good.
> > > > 
> > > > >        'filter' => 'wikicontent',
> > > > 
> > > > is which filter is applied to the input data, and wikicontent is probably
> > > > fine here, it means anything you would put into a wiki page (but it would
> > > > be just text, a pagename or a username etc, again, probably not terribly
> > > > well documented)
> > > > 
> > > > >        'tags' => [ 'basic' ],
> > > > 
> > > > Is should the pref for this appear in basic, advanced or even experimental
> > > > views (in the control panels) - i'm not so sure that actually has much
> > > > effect on plugins these days...
> > > > 
> > > > One that is missing is the 'format' property (which seems to be missing
> > > > from wikiplugin_div too). This defaults to 'wiki' but sometimes should be
> > > > set to 'html' if there is no further parsing of the output required,
> > > > whereas here you probably have more parsing to do on the $data in case
> > > > there is formatting or something to be done on what's being indented.
> > > 
> > > But the 'format' property is there. Have you overlooked it?
> > > 
> > > > So good to go, i think, well done!
> > > > 
> > > > Next, https://gitlab.com/tikiwiki/tiki/-/merge_requests/new :)
> > > 
> > > This produces a "Page not found" error.
> > > 
> > > I'm doing a merge request now.
> > > 
> > > Cheers, Volker
> > > 
> > > > jonny
> > > > 
> > > > 
> > > > 
> > > > 
> > > > 
> > > > > On 10 Dec 2021, at 17:56, Volker Wysk <post-hhF2Jplw28UoZk/[email protected]> wrote:
> > > > > 
> > > > > Hi!
> > > > > 
> > > > > I made a simple plugin, which allows for the indentation of arbitrary blocks
> > > > > of wiki content, not just inside of lists. It's a feature which MoinMoin
> > > > > has, which (it seems) is not present in Tiki so far. I'd like it to be
> > > > > included in Tiki 24.
> > > > > 
> > > > > It's so small that I include it here:
> > > > > 
> > > > > -----snip-----
> > > > > <?php
> > > > > // ... copyright notice ...
> > > > > function wikiplugin_indent_info()
> > > > > {
> > > > >    return [
> > > > >        'name' => tra('Indentation'),
> > > > >        'documentation' => 'PluginIndent',
> > > > >        'description' => 
> > > > >            tra('Indent a block of wiki content by one level.'),
> > > > >        'format' => 'wiki',
> > > > >        'prefs' => [ 'wikiplugin_indent' ],
> > > > >        'body' => tra('Wiki content (text) that is to be indented.'),
> > > > >        'introduced' => 24,
> > > > >        'filter' => 'wikicontent',
> > > > >        'tags' => [ 'basic' ],
> > > > >        'params' => [ ],
> > > > >    ];
> > > > > }
> > > > > 
> > > > > function wikiplugin_indent($data, $params)
> > > > > {
> > > > >    return "<ul><li style=\"list-style-type:none\">$data</li></ul>";
> > > > > }
> > > > > -----snip-----
> > > > > 
> > > > > I've taken PluginSup as a starting point. 
> > > > > 
> > > > > Now there are two points which need to be addressed before I can do a merge
> > > > > request.
> > > > > 
> > > > > - Please have a look at the plugin properties (above). The properties
> > > > > 'introduced', 'filter' and 'tags' are guesswork. I've taken them from
> > > > > PluginSup. They aren't documented in any of these places:
> > > > > 
> > > > > https://dev.tiki.org/Hello+World#To_create_a_new_plugin
> > > > > https://dev.tiki.org/Code-Howto%3A-Create-a-Wiki-Plugin
> > > > > 
> > > > > - I'd like the wikiplugin_indent to be enabled by default. Is this okay?
> > > > > 
> > > > > 
> > > > > Cheers,
> > > > > Volker
> > > > > _______________________________________________
> > > > > TikiWiki-devel mailing list
> > > > > [email protected]
> > > > > https://lists.sourceforge.net/lists/listinfo/tikiwiki-devel
> > > > 
> > > > _______________________________________________
> > > > TikiWiki-devel mailing list
> > > > [email protected]
> > > > https://lists.sourceforge.net/lists/listinfo/tikiwiki-devel
> > > 
> > > _______________________________________________
> > > TikiWiki-devel mailing list
> > > [email protected]
> > > https://lists.sourceforge.net/lists/listinfo/tikiwiki-devel
> > 
> > _______________________________________________
> > TikiWiki-devel mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/tikiwiki-devel
> 
> _______________________________________________
> TikiWiki-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/tikiwiki-devel

_______________________________________________
TikiWiki-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tikiwiki-devel
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEE6QXGh82Ov3+2nrxp+K4ydFOsHoUFAmG2S8QACgkQ+K4ydFOs
HoWTmQ//X8H5kAnOJWmyU83XOntJOoITQ6M0KdIATCFDoGM+XSauOcgLHHizKZD4
Uqr+PS0gjTQLfjGI+NxursP7BgI0ldb+0C7oFi1yrLTt+OzrhZ0yUQZFlAtGkntu
6ZEy7XuMYCdeih4GqVHxLRAAz2OlclRCewujbSTy+sapEdmo+cOCC9Bn+4h9yzzp
RVL/kXyICl8cB8GK8ZO+AO4Po0IQzcx576ojU7Ovxflf/h4O8DQU8AkryqxOo81v
SjCMFiGYgbhi1V5u0AVuvq5PRef7BTVz8nCzYFKMcxqIOdYWOdq1ye0Um9txWIXM
hH1OxITx5WhOdu32tQedrIz4Vh9ZC4AXuUWwU/BrL+uPx9LCyhByyUY1qMOu4tTQ
zDACGyr3F7acW0LPfSfhvpi/UkvkG6GgSdEiUsM0UGanObURM8RrQ263b5v/zYp6
bVgCDFx8+IUDVkc8QEc7+qtsBJ1BsDKaDEwofpesW4yfo08dWkQBFI2vh0u38Zra
60mF/JfZt68WJagd35Z8zCNf6RWGlUmn/Mn6tHDaAUJjIUv1SuWflu6PBgTRUv4t
KV5jnadvZ+Wien6afTkwd+8TO0+KnZ5yYDLe9UYAGJ5s9Jw5UzgXUz8GDxbO1xaG
ivnRFwQ0LS4kEiCxABZbzBJ3kfLBryy7OySM1a42YwE4e+M2U/M=
=31rf
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.