@edwardspec, who can review the patch?

Thank you!

it works, tested on 1.35.0 (9b8a168)

In T267213#6669371, @ssastry wrote:

FWIW, I've worked with @Pastakhov of WikiTeq during the porting of Parsoid from JS to PHP and found them competent and well-qualified.

ALTER TABLE user CONVERT TO CHARACTER SET binary; helped me

Fixed by https://gerrit.wikimedia.org/r/c/mediawiki/extensions/PhpTags/+/610343

done

I think it is a good idea, 1.0 :)
thanks for the contributions!

Hi Yaron, I saw it, just mark "Set Ready For Review" when they ready and I'll +2 them.
I get notifications about all new patches, but ping me if I don't answer for a long time. Unfortunately, sometimes I may not notice something, sorry

I missed the patch, thanks for the ping

yes, I'm one, I'll fix it when I get some time

Hi, I still plan to work on this task

Hi, I still plan to work on this task

The file was uploaded with empty Summary field, I tried to rename (move) another file that was loaded with empty Summary field and I got the error.
how it looks in the database:

mysql> SELECT t.* FROM page t WHERE page_id = 99;
+---------+----------------+-------------------+-------------------+------------------+-------------+----------------+----------------+-------------+----------+--------------------+--------------------+-----------+
| page_id | page_namespace | page_title        | page_restrictions | page_is_redirect | page_is_new | page_random    | page_touched   | page_latest | page_len | page_content_model | page_links_updated | page_lang |
+---------+----------------+-------------------+-------------------+------------------+-------------+----------------+----------------+-------------+----------+--------------------+--------------------+-----------+
|      99 |              6 | KarlMarxJeune.jpg |                   |                0 |           1 | 0.621428611495 | 20190113233540 |         281 |        0 | wikitext           | 20180410093726     | NULL      |
+---------+----------------+-------------------+-------------------+------------------+-------------+----------------+----------------+-------------+----------+--------------------+--------------------+-----------+
1 row in set (0.00 sec)

I just successfully moved the file page after I adding text on the page, I think you don't face the issue because, as usual, all files contain text on wikipedia's file pages

debug info P9459

It seems this bug is not resolved yet.
I faced a similar problem on 1.33.0, when I'm trying to move page I got the error:

[0bff6a9515dc740c4d9e9790] /wr/index.php?title=Sp%C3%A9cial:Renommer_une_page&action=submit MediaWiki\Revision\IncompleteRevisionException from line 425 of /opt/mediawiki-1.33.1/includes/Revision/RevisionStore.php: sha1 field must not be ''!

In T69074#737261, @Rillke wrote:

after increasing memory to 204800 and restarting apache, it no more failed thumbnails for that reason -- still needs investigation

There is a few more errors because PHP 7.1 is more strictly related to the type of data, it is good but I need to process these incidents inside phptags too,
for example
php5.5 php7.0:
["4"] & "fff"; is ok
php 7.1
["4"] & "fff";

PHP Warning:  A non-numeric value encountered in Command line code on line 1

@hashar, thanks for looking at the failures. I debugged the problem, it happens because PHP 7 assigns the values in the list operator in reverted order (link).
I'll fix it soon

please, check it https://gerrit.wikimedia.org/r/#/c/435971/

Oh, sorry for delay, I try to update it right now

Oh, sorry. I got hard busy... I hope to continue in 2-3 weeks :-/

! In T174145#3652265, @kaldari wrote:
Note that we have to use an OOjs-UI icon for syntax highlighting within the 2017 Wikitext editor. That interface is constructed using OOjs-UI, so the only choices we have are the icons in https://github.com/wikimedia/oojs-ui/tree/master/src/themes/wikimediaui/images/icons.

looks like a virus :)

Seems should be fixed. Should be used different keyMaps for Mac and PC.

the path was rebased to the latest version of CodeMirror where the mentioned bug is fixed.

The status was changed by mistake... I did not want to do it by myself

ctrl + arrow key still working. Change 373624 makes behavior of alt + arrow key the same as when textarea used, see bug T174055.

Home / End keys cumbersome behavior

I think there should not be a big problem to make it resizable, and there should not be a problem with performance. I'll try to fix it.

can't reproduce on en.wikipedia.org

@Niharika yes, I understand about cost. I would prefer to find a balance between speed and ease of use.

Seems should be fixed with T174547

Should be merged with T174514

There is a bug: Home after End goes to the start of the next wrapped line. Hopefully it was fixed in latest CodeMirror library.

Solution is Better Home/End navigation with wrapped lines? #1576

The bug in the CodeMirror library Paste by using middle click in firefox not working #931

Maybe try to set viewportMargin to Infinity when ctrl+F pressed?

As a variant the patch above calls the CodeMirror.enabled and CodeMirror.disabled hooks.
Catching ones gets you full access to the CodeMirror features.
For example, alert message when the text is changed:

mediaWiki.hook( 'CodeMirror.enabled' ).add( 
    function ( cm ) { 
        cm.on( 'change', function() { 
            alert ( 'text changed' ); 
        } );
    } 
);

It used event change of CodeMirror.