Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug fix 1.7.3: SQL error when saving a module #32

Closed
wants to merge 2 commits into from
Closed

Bug fix 1.7.3: SQL error when saving a module #32

wants to merge 2 commits into from

Conversation

eddieajau
Copy link
Contributor

Saving a module produces an SQL error due to incorrect usage of the query builder. JDatabaseQuery::values takes a whole tuple of values or an array of tuples (which differs from the columns method which can add to the list of columns with subsequent calls).

Patch also adds some other common files to .gitignore.

NOTE: Error is occurring against the master git platform, not Joomla's copy of /libraries/joomla/.

@dextercowley
Copy link
Contributor

Andrew, is there a tracker issue associated with this? Thanks. Mark

On Thu, Dec 8, 2011 at 1:42 AM, Andrew Eddie
reply@reply.github.com
wrote:

Saving a module produces an SQL error due to incorrect usage of the query builder.  JDatabaseQuery::values takes a whole tuple of values or an array of tuples (which differs from the columns method which can add to the list of columns with subsequent calls).

Patch also adds some other common files to .gitignore.

You can merge this Pull Request by running:

 git pull https://github.com/eddieajau/joomla-cms bugs

Or you can view, comment on it, or merge it online at:

 #32

-- Commit Summary --

  • Fix SQL when trying to save a module due to query build error.
  • Add extra ignore filters for common files.

-- File Changes --

M .gitignore (6)
M administrator/components/com_modules/models/module.php (5)

-- Patch Links --

 https://github.com/joomla/joomla-cms/pull/32.patch
 https://github.com/joomla/joomla-cms/pull/32.diff


Reply to this email directly or view it on GitHub:
#32

@eddieajau
Copy link
Contributor Author

No. The problem is actually more widespread. I'll ping you about it.

elinw referenced this pull request in elinw/joomla-cms Dec 11, 2011
Cleaning up after JDocument changes
@eddieajau
Copy link
Contributor Author

Closing.

@eddieajau eddieajau closed this Dec 12, 2011
wilsonge referenced this pull request in wilsonge/joomla-cms Aug 29, 2013
Fix "Error displaying the error page: Application Instantiation Error"
wilsonge pushed a commit that referenced this pull request Apr 7, 2016
wilsonge pushed a commit that referenced this pull request Jan 23, 2017
wilsonge pushed a commit that referenced this pull request May 31, 2017
* codestyle

* code style

* codestyle

* codestyle

* codestyle

* thanks @wojsmol

* corrections - thanks @Quy

* corrections - thanks @Quy

* oops

* make @Quy happy

* Update article.xml

* Remove space

* Update config.xml (#14)

* Update filter.xml (#15)

* Update config.xml (#16)

* Update profile.xml (#17)

* Update application.xml (#18)

* Update article.xml (#19)

* Update filter_articles.xml (#20)

* Update config.xml (#24)

* Update config.xml (#23)

* Update filter_fields.xml (#22)

* Update filter_featured.xml (#21)

* Update override.xml (#25)

* Update config.xml

* Update config.xml (#26)

* Update itemadmin_alias.xml (#30)

* Update itemadmin.xml (#29)

* Update item.xml (#27)

* Update item_alias.xml (#28)

* Update itemadmin_url.xml (#31)

* Update module.xml (#32)

* Update plugin.xml (#33)

* Update config.xml (#34)

* Update link.xml (#35)

* Update config.xml (#36)

* Update style.xml (#38)

* Update config.xml (#37)

* Update note.xml (#42)

* Update group.xml (#41)

* Update filter_debuggroup.xml (#40)

* Update config.xml (#39)

* corrections for @andrepereiradasilva

* gotya
bembelimen referenced this pull request in bembelimen/joomla-cms Apr 26, 2018
wilsonge pushed a commit that referenced this pull request Aug 15, 2019
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.

2 participants