Skip to content

Removing duplicated AS in sql query#13006

Merged
Bakual merged 1 commit intojoomla:stagingfrom
sweco-semhul:sql_server_smart_search_remove_link_error
Nov 25, 2016
Merged

Removing duplicated AS in sql query#13006
Bakual merged 1 commit intojoomla:stagingfrom
sweco-semhul:sql_server_smart_search_remove_link_error

Conversation

@sweco-semhul
Copy link
Copy Markdown
Contributor

Pull Request for Issue #13005.

Summary of Changes

Removing duplicated AS in sql query resolving query problem

Testing Instructions

Setup Joomla against SQL Server
Go to "smart search"
Reindex
Select any link in the list and click Delete

Documentation Changes Required

none

@brianteeman
Copy link
Copy Markdown
Contributor

(you don't need to create an issue if you are going to create a pull request - the system is clever and all pull requests appear in the issue tracker automatically)

@sweco-semhul
Copy link
Copy Markdown
Contributor Author

ok. since the PR template included issue information I thought it was mandatory

@brianteeman
Copy link
Copy Markdown
Contributor

brianteeman commented Nov 24, 2016 via email

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Nov 24, 2016

I have tested this item ✅ successfully on 343a929

on code review


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

@bembelimen
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 343a929


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

@Bakual Bakual merged commit 0c0884f into joomla:staging Nov 25, 2016
@Bakual Bakual added this to the Joomla 3.7.0 milestone Nov 25, 2016
roland-d added a commit to roland-d/joomla-cms that referenced this pull request Nov 25, 2016
* staging: (98 commits)
  Coding style. PHP constants true, false, and null MUST be in lower case. (joomla#13010)
  Removing duplicated AS in sql query (joomla#13006)
  Fixed typo in comment (joomla#12992)
  Correcting strings in TFA Google plugin (joomla#12980)
  code style changes (joomla#12986)
  Error in sr-YU installation ini file (joomla#12984)
  New DateTime picker (replaces calendar) (joomla#11138)
  Export of Banners Tracks Does Not Export the Banner Name
  fix rues get data (joomla#12763)
  Added Feature items filter to mod_articles_news (joomla#12547)
  fix them all (joomla#12943)
  a11y regression fix (joomla#12935)
  Set correct component id for system links (joomla#12938)
  Fix for Undefined offset in Content History preview popup (joomla#12791)
  remove tab on meta charset (joomla#12895)
  JSession patched to set session _state to 'inactive' when session is closed. (joomla#12928)
  [JHtmlNumber::bytes] Format number according to language (joomla#12929)
  Update edit.php (joomla#12818)
  Update default.xml (joomla#12917)
  Adding the ability to use the global value for character count in newsfeeds (joomla#12869)
  ...
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.

6 participants