? Success

User tests: Successful: Unsuccessful:

avatar okonomiyaki3000
okonomiyaki3000
9 Sep 2014

Steps to reproduce the issue

There's no issue. Just an update to CodeMirror.

Expected result

CodeMirror will be v4.5 as seen here http://codemirror.net/doc/releases.html

Actual result

System information (as much as possible)

avatar okonomiyaki3000 okonomiyaki3000 - open - 9 Sep 2014
avatar jissues-bot jissues-bot - change - 9 Sep 2014
Labels Added: ?
avatar okonomiyaki3000
okonomiyaki3000 - comment - 9 Sep 2014

What's going on with these build failures?

avatar zero-24
zero-24 - comment - 9 Sep 2014

@okonomiyaki3000

What's going on with these build failures?

Not all related to this PR. There are some deprecated Messsages like PHPUnit_Framework_Assert::assertTag is deprecated

and on CS Issue that Travis find

FILE: .../joomla-cms/components/com_config/view/modules/tmpl/default_options.php
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 27 | ERROR | Increment and decrement operators must be bracketed when used in
    |       | string concatenation
--------------------------------------------------------------------------------
avatar zero-24 zero-24 - change - 9 Sep 2014
Easy No Yes
avatar zero-24 zero-24 - change - 9 Sep 2014
Category External Library Plugins
avatar zero-24
zero-24 - comment - 9 Sep 2014

@test successful install and testing your branche https://github.com/okonomiyaki3000/joomla-cms/archive/CodeMirrorUpdate.zip

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

avatar okonomiyaki3000
okonomiyaki3000 - comment - 9 Sep 2014

I agree those are weird. Looks like wrong encoding but, upon closer inspection, I think it's actually supposed to be that way. In any case, they're irrelevant. Those files are included in the CodeMirror package but they are just html files and not related to the use of CodeMirror in anyway. They are working demos of each mode. We will never use them but it's troublesome to remove them all from the package.

avatar zero-24
zero-24 - comment - 28 Sep 2014

@test still successfull using https://github.com/okonomiyaki3000/joomla-cms/archive/CodeMirrorUpdate.zip

BTW: the ��� are fixed with the last commits :)

Thanks @okonomiyaki3000 :+1:

avatar infograf768
infograf768 - comment - 29 Sep 2014

Works OK for me

avatar roland-d roland-d - alter_testresult - 3 Oct 2014 - infograf768: Tested successfully
avatar roland-d roland-d - change - 3 Oct 2014
Status Pending Ready to Commit
avatar phproberto phproberto - change - 8 Oct 2014
Labels Added: ?
avatar mbabker mbabker - close - 8 Oct 2014
avatar mbabker mbabker - change - 8 Oct 2014
Title
CodeMirror 4.5
CodeMirror 4.6
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-10-08 14:19:35

Add a Comment

Login with GitHub to post a comment