?
avatar gunjanpatel
gunjanpatel
29 Feb 2016

Not sure if it is a bug or not but found interesting.

Steps to reproduce the issue

While installing component I notice that my queries doesn't work in j3.5-beta3.

Query Working in J3.4x

INSERT IGNORE INTO `#__redshop_mail` (`mail_id`, `mail_body`) VALUES
    (1, '<p>To Admin, {username} </p>');

The same query is not working in J3.5-beta3. While I change query as shown below it works.

INSERT IGNORE INTO `#__redshop_mail` (`mail_id`, `mail_body`) VALUES
    (1, '&lt;p&gt;To Admin,&lt;/p&gt;');

To make it working in j3.5-beta3 I have to change,
'<p>To Admin,</p>' to '&lt;p&gt;To Admin, {username} &lt;/p&gt;'

Not sure what is changed in j3.5-beta3. But the same query works in j3.4.x and if I paste the query directly in mysql it works.

Votes

# of Users Experiencing Issue
2/2
Average Importance Score
5.00

avatar gunjanpatel gunjanpatel - open - 29 Feb 2016
avatar javigomez javigomez - reference | c508e4f - 29 Feb 16
avatar andrepereiradasilva
andrepereiradasilva - comment - 29 Feb 2016
avatar gunjanpatel
gunjanpatel - comment - 1 Mar 2016

@andrepereiradasilva it works with 3.5.0 beta 2. Only issue in 3.5 beta 3


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9254.

avatar andrepereiradasilva
andrepereiradasilva - comment - 1 Mar 2016

just guessing maybe because of a change that was corrected today (see fc43245)

Can you check if your issue remains in latest staging
https://github.com/joomla/joomla-cms/archive/staging.zip

avatar wilsonge
wilsonge - comment - 2 Mar 2016

I don't think so - that issue was specifically dealing with hashtags that aren't present in the SQL file dump provided. I need some more information please @gunjanpatel - I don't really have enough here to work with

avatar flash1452
flash1452 - comment - 2 Mar 2016

I don't think it should happen in HTML because i have had experience of such errors in XML,not with HTML tags.

avatar gunjanpatel
gunjanpatel - comment - 3 Mar 2016

@wilsonge I am going to attach installer in which I am having issue. May be it will help to reproduce it.
HelloWorldj3.5.beta3

avatar javigomez
javigomez - comment - 3 Mar 2016

I can confirm that it works in Joomla 3.5.0-beta2:

screen shot 2016-03-03 at 10 20 44

While fails in Joomla 3.5.0-beta3:

screen shot 2016-03-03 at 10 21 13

avatar infograf768
infograf768 - comment - 3 Mar 2016

I could install OK HelloWorldj3.5.beta3 on staging (localhost)

avatar javigomez
javigomez - comment - 3 Mar 2016

I have been able to isolate the problem. The issue happens when using a # symbol in the sql. This fails:

data.sql:

INSERT IGNORE INTO `#__helloworld_mail` (`mail_id`, `mail_name`, `mail_subject`, `mail_section`, `mail_order_status`, `mail_body`, `published`, `mail_bcc`) VALUES
    (24, 'Order Status Change Pending', 'Order Status Change Pending', 'order_status', 'P', '#', 1, ''),
    (25, 'Order Status Change Confirmed', 'Order Status Change Confirmed', 'order_status', 'C', '#', 1, '');

But if you remove the # at ...'order_status', 'P', '#', 1, ''),... and at ...er_status', 'C', '#', 1, '');

It works

I'm attaching the example component refactored:

com_helloworld.zip

avatar javigomez
javigomez - comment - 3 Mar 2016

I confirm @infograf768 comment the problem does not happen in staging:

screen shot 2016-03-03 at 11 19 16

It appears only testing against https://github.com/joomla/joomla-cms/releases/tag/3.5.0-beta3

avatar brianteeman
brianteeman - comment - 3 Mar 2016

Then this can be closed then as it has already been fixed

On 3 March 2016 at 10:20, javier gomez notifications@github.com wrote:

I confirm @infograf768 https://github.com/infograf768 comment the
problem does not happen in staging:

[image: screen shot 2016-03-03 at 11 19 16]
https://cloud.githubusercontent.com/assets/1375475/13491269/d4bf0030-e131-11e5-9cd4-8d64f2722d86.png

It appears only testing against
https://github.com/joomla/joomla-cms/releases/tag/3.5.0-beta3


Reply to this email directly or view it on GitHub
#9254 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar ot2sen
ot2sen - comment - 3 Mar 2016

Adding the fix for driver.php from fc43245 do solve the issue it seems

avatar brianteeman brianteeman - change - 3 Mar 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-03-03 10:44:38
Closed_By brianteeman
avatar brianteeman brianteeman - close - 3 Mar 2016

Add a Comment

Login with GitHub to post a comment