Skip to content

[PostgreSQL - MSSQL] sql error editing module or template#8236

Merged
wilsonge merged 1 commit intojoomla:stagingfrom
alikon:nomysql-1
Nov 2, 2015
Merged

[PostgreSQL - MSSQL] sql error editing module or template#8236
wilsonge merged 1 commit intojoomla:stagingfrom
alikon:nomysql-1

Conversation

@alikon
Copy link
Copy Markdown
Contributor

@alikon alikon commented Nov 2, 2015

Steps to reproduce the issue

administration->extensions->modules edit a module
or/and
administration->extensions->templates edit a template

Actual result

module
j35 administration modules articles latest
template
j35 administration templates edit style

After patch

able to edit a module or template

@waader
Copy link
Copy Markdown
Contributor

waader commented Nov 2, 2015

I have tested this item ✅ successfully on 6747067

Something I observed while testing this patch on mssql: when you edit a module and change for example the position and then save, after reentering the module, the changed value is not present any more. It works however for the template styles.

Maybe you can have a look at this.


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

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Nov 2, 2015

I have tested this item ✅ successfully on 6747067

Works on MySQL after that patch. Thanks. 👍


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

@zero-24 zero-24 added this to the Joomla! 3.5.0 milestone Nov 2, 2015
@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Nov 2, 2015

RTC. Great!


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 2, 2015
wilsonge added a commit that referenced this pull request Nov 2, 2015
[PostgreSQL -  MSSQL] sql error editing module or template
@wilsonge wilsonge merged commit 8378513 into joomla:staging Nov 2, 2015
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants