?

User tests: Successful: Unsuccessful:

avatar chrisdavenport
chrisdavenport
14 Aug 2014
avatar chrisdavenport chrisdavenport - open - 14 Aug 2014
avatar jissues-bot jissues-bot - change - 14 Aug 2014
Status Pending New
Labels Added: ? ?
avatar zero-24
zero-24 - comment - 14 Aug 2014

@chrisdavenport It looks that github think your pull request contains merge conflicts.

avatar infograf768
infograf768 - comment - 15 Aug 2014

This can't be merged.

avatar chrisdavenport
chrisdavenport - comment - 15 Aug 2014

Any idea what the problem is and how to fix it?

avatar infograf768
infograf768 - comment - 15 Aug 2014

You should update your branch to present 2.5.x, and then patch again
I did it manually for contactemail here and I get the following diff. See the difference in the lines:
@@ -23,5 +23,5 @@ vs @@ -22,7 +22,7 @@

diff --git a/components/com_contact/models/rules/contactemail.php b/components/com_contact/models/rules/contactemail.php
index 5904981..a910c01 100644
--- a/components/com_contact/models/rules/contactemail.php
+++ b/components/com_contact/models/rules/contactemail.php
@@ -23,5 +23,5 @@

        foreach(explode(';', $banned) as $item){
-           if ($item != '' && JString::stristr($item, $value) !== false)
+           if (JString::stristr($value, $item) !== false)
                    return false;
        }

while your PR has:

diff --git a/components/com_contact/models/rules/contactemail.php b/components/com_contact/models/rules/contactemail.php
index 875b1e1..bf34f09 100644
--- a/components/com_contact/models/rules/contactemail.php
+++ b/components/com_contact/models/rules/contactemail.php
@@ -22,7 +22,7 @@ public function test(& $element, $value, $group = null, & $input = null, & $form
        $banned = $params->get('banned_email');

        foreach(explode(';', $banned) as $item){
-           if (JString::stristr($item, $value) !== false)
+           if (JString::stristr($value, $item) !== false)
                    return false;
        }
avatar chrisdavenport
chrisdavenport - comment - 15 Aug 2014

What I did (from memory) is:
On GitHub I forked joomla-cms
On CLI:
git clone
git remote add upstream https://github.com/joomla/joomla-cms.git
git checkout 2.5.x
[made changes to the code]
git add
git commit
git push

What did I miss?

avatar chrisdavenport
chrisdavenport - comment - 16 Aug 2014

See #4120 for second attempt.

avatar brianteeman
brianteeman - comment - 16 Aug 2014

closing in favour of #4120

avatar brianteeman brianteeman - change - 16 Aug 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-08-16 10:37:27
avatar brianteeman brianteeman - close - 16 Aug 2014

Add a Comment

Login with GitHub to post a comment