Page 2 of 5

Re: [3.1][3.2][RC] Twig Converter

Posted: Sat Jul 01, 2017 11:23 pm
by 3Di
Finally I got 10 minutes of spare time to d/l the latest and test it.

On a side note: the PHP extension php_fileinfo is a requirement here, lesson taken and server re-started. ;) Maybe add a prior check in ext.php, kasimi?

Going to test this with a mix-salad (half and half template syntaxes). I will report.

Edit: success, the only ACP page left to be translated, mixed-salad, has been succesfully rewritten, also some commented code has been translated. Page tested as well. At a first glance, you got a winner. Thanks. :)

Re: [3.1][3.2][RC] Twig Converter

Posted: Sun Jul 02, 2017 8:20 am
by kasimi
Thanks for testing. With these changes the php_fileinfo extension shouldn't be needed anymore.

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 2:42 pm
by </Solidjeuh>
I have these errors when I convert my templates:

Code: Select all

[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 550: Undefined index: ulist_open_default
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 550: Undefined index: ulist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 553: Undefined index: olist_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 553: Undefined index: olist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 556: Undefined index: ulist_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 556: Undefined index: ulist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 560: Undefined index: listitem
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 560: Undefined index: listitem_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 570: Undefined index: inline_attachment_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 570: Undefined index: inline_attachment_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 550: Undefined index: ulist_open_default
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 550: Undefined index: ulist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 553: Undefined index: olist_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 553: Undefined index: olist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 556: Undefined index: ulist_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 556: Undefined index: ulist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 560: Undefined index: listitem
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 560: Undefined index: listitem_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 570: Undefined index: inline_attachment_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 570: Undefined index: inline_attachment_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 550: Undefined index: ulist_open_default
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 550: Undefined index: ulist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 553: Undefined index: olist_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 553: Undefined index: olist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 556: Undefined index: ulist_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 556: Undefined index: ulist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 560: Undefined index: listitem
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 560: Undefined index: listitem_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 570: Undefined index: inline_attachment_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 570: Undefined index: inline_attachment_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 550: Undefined index: ulist_open_default
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 550: Undefined index: ulist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 553: Undefined index: olist_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 553: Undefined index: olist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 556: Undefined index: ulist_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 556: Undefined index: ulist_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 560: Undefined index: listitem
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 560: Undefined index: listitem_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 570: Undefined index: inline_attachment_open
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 570: Undefined index: inline_attachment_close
[phpBB Debug] PHP Notice: in file [ROOT]/phpbb/textformatter/s9e/factory.php on line 505: Undefined index: code
phpBB: 3.2.1-RC1
PHP: 7.0.12

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 4:30 pm
by kasimi
Your style's bbcode.html is missing or incomplete.

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 4:38 pm
by </Solidjeuh>
kasimi wrote:
Mon Jul 03, 2017 4:30 pm
Your style's bbcode.html is missing or incomplete.
This is what I got from the conversion

Code: Select all

{% for ulist_open in loops.ulist_open %}<ul style="list-style-type: {{ LIST_TYPE }}">{% endfor %}
{% for ulist_open_default in loops.ulist_open_default %}<ul>{% endfor %}
{% for ulist_close in loops.ulist_close %}</ul>{% endfor %}

{% for olist_open in loops.olist_open %}<ol style="list-style-type: {{ LIST_TYPE }}">{% endfor %}
{% for olist_close in loops.olist_close %}</ol>{% endfor %}

{% for listitem in loops.listitem %}<li>{% endfor %}
{% for listitem_close in loops.listitem_close %}</li>{% endfor %}

{% for quote_username_open in loops.quote_username_open %}<blockquote><div><cite>{{ USERNAME }} {{ lang('WROTE') }}{{ lang('COLON') }}</cite>{% endfor %}
{% for quote_open in loops.quote_open %}<blockquote class="uncited"><div>{% endfor %}
{% for quote_close in loops.quote_close %}</div></blockquote>{% endfor %}
{% for quote_extended in loops.quote_extended %}
<blockquote>
	<xsl:if test="not(@author)">
		<xsl:attribute name="class">uncited</xsl:attribute>
	</xsl:if>
	<div>
		<xsl:if test="@author">
			<cite>
				<xsl:choose>
					<xsl:when test="@url">
						<a href="{@url}" class="postlink"><xsl:value-of select="@author"/></a>
					</xsl:when>
					<xsl:when test="@profile_url">
						<a href="{@profile_url}"><xsl:value-of select="@author"/></a>
					</xsl:when>
					<xsl:otherwise>
						<xsl:value-of select="@author"/>
					</xsl:otherwise>
				</xsl:choose>
				<xsl:text> </xsl:text>
				<xsl:value-of select="$L_WROTE"/>
				<xsl:value-of select="$L_COLON"/>
				<xsl:if test="@post_url">
					<xsl:text> </xsl:text>
					<a href="{@post_url}" data-post-id="{@post_id}" onclick="if(document.getElementById(hash.substr(1)))href=hash">&#8593;</a>
				</xsl:if>
				<xsl:if test="@date">
					<div class="responsive-hide"><xsl:value-of select="@date"/></div>
				</xsl:if>
			</cite>
		</xsl:if>
		<xsl:apply-templates/>
	</div>
</blockquote>
{% endfor %}

{% for code_open in loops.code_open %}<div class="codebox"><p>{{ lang('CODE') }}{{ lang('COLON') }} <a href="#" onclick="selectCode(this); return false;">{{ lang('SELECT_ALL_CODE') }}</a></p><pre><code>{% endfor %}
{% for code_close in loops.code_close %}</code></pre></div>{% endfor %}

{% for inline_attachment_open in loops.inline_attachment_open %}<div class="inline-attachment">{% endfor %}
{% for inline_attachment_close in loops.inline_attachment_close %}</div>{% endfor %}

{% for b_open in loops.b_open %}<strong class="text-strong">{% endfor %}
{% for b_close in loops.b_close %}</strong>{% endfor %}

{% for u_open in loops.u_open %}<span style="text-decoration: underline">{% endfor %}
{% for u_close in loops.u_close %}</span>{% endfor %}

{% for i_open in loops.i_open %}<em class="text-italics">{% endfor %}
{% for i_close in loops.i_close %}</em>{% endfor %}

{% for color in loops.color %}<span style="color: {{ COLOR }}">{{ TEXT }}</span>{% endfor %}

{% for size in loops.size %}<span style="font-size: {{ SIZE }}%; line-height: 116%;">{{ TEXT }}</span>{% endfor %}

{% for img in loops.img %}<img src="{{ URL }}" class="postimage" alt="{{ lang('IMAGE') }}" />{% endfor %}

{% for url in loops.url %}<a href="{{ URL }}" class="postlink">{{ DESCRIPTION }}</a>{% endfor %}

{% for email in loops.email %}<a href="mailto:{{ EMAIL }}">{{ DESCRIPTION }}</a>{% endfor %}

{% for flash in loops.flash %}<object classid="clsid:D27CDB6E-AE6D-11CF-96B8-444553540000" codebase="http://active.macromedia.com/flash2/cabs/swflash.cab#version=5,0,0,0" width="{{ WIDTH }}" height="{{ HEIGHT }}"><param name="movie" value="{{ URL }}" /><param name="play" value="false" /><param name="loop" value="false" /><param name="quality" value="high" /><param name="allowScriptAccess" value="never" /><param name="allowNetworking" value="internal" /><embed src="{{ URL }}" type="application/x-shockwave-flash" pluginspage="http://www.macromedia.com/shockwave/download/index.cgi?P1_Prod_Version=ShockwaveFlash" width="{{ WIDTH }}" height="{{ HEIGHT }}" play="false" loop="false" quality="high" allowscriptaccess="never" allownetworking="internal"></embed></object>{% endfor %}

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 4:56 pm
by kasimi
It looks like the textformatter expects bbcode.html to use phpBB template syntax: https://github.com/phpbb/phpbb/blob/rel ... y.php#L508

Here's an idea: before this line, add this code:

Code: Select all

if ($template_file['filename'] == 'bbcode.html')
{
    continue;
}
Maybe JoshyPHP has details on this.
JoshyPHP wrote: poke

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 4:58 pm
by JoshyPHP
Yes, everything in phpBB expects bbcode.html to use the <!-- BEGIN ... --> syntax.

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 5:27 pm
by </Solidjeuh>
kasimi wrote:
Mon Jul 03, 2017 4:56 pm
It looks like the textformatter expects bbcode.html to use phpBB template syntax: https://github.com/phpbb/phpbb/blob/rel ... y.php#L508

Here's an idea: before this line, add this code:

Code: Select all

if ($template_file['filename'] == 'bbcode.html')
{
    continue;
}
Maybe JoshyPHP has details on this.
JoshyPHP wrote: poke
Yes that worked, no more errors :D

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 8:02 pm
by kasimi
Thank you both for confirming.

Changes in version 1.1.1:
  • When downloading zip file, set content type explicitly to not rely on php_fileinfo extension
  • When converting a style, don't convert bbcode.html
For the latest download see the link to the GitHub releases in the first post.

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 8:13 pm
by Toxyy
Does twig syntax work on the ACP? Did it by hand the other day and it broke, wasn't sure if it was something on my end or not.

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 8:17 pm
by david63
Toxyy wrote:
Mon Jul 03, 2017 8:13 pm
Does twig syntax work on the ACP?
Yes

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 9:02 pm
by </Solidjeuh>
Toxyy wrote:
Mon Jul 03, 2017 8:13 pm
Does twig syntax work on the ACP? Did it by hand the other day and it broke, wasn't sure if it was something on my end or not.
Yes just tried it. Everything works.

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 9:29 pm
by 3Di
What about trying first and in case provide feedback..
that's still a developmental topic, therefore is provided for testing.

The ACP side has been tested already, with mixed-salad as well
viewtopic.php?p=14772336#p14772336

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 9:44 pm
by Toxyy
Can there be an option to converted to 3.2 twig? The only change there would be removing "loops." from for statements.

Re: [3.1][3.2][RC] Twig Converter

Posted: Mon Jul 03, 2017 9:52 pm
by 3Di
loops. are compatible.