Are you sure removing the OR statement is appropriate? I can't tell right
now because I'm on mobile, but IIRC, that's needed because the query is
being built in a foreach loop.

On Thursday, April 17, 2014, zero-24 notifications@github.com wrote:

Tracker:

http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33626&start=0

You can merge this Pull Request by running

git pull https://github.com/zero-24/joomla-cms patch-17

Or view, comment on, or merge it at:

#3471
Commit Summary

• Fix failing manifest updates

File Changes

• M administrator/components/com_admin/script.phphttps://github.com/joomla/joomla-cms/pull/3471/files#diff-0(2)

Patch Links:

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

—
Reply to this email directly or view it on GitHub#3471
.

@mbabker
ok. But least the last stament is wrong as it contains a "or" without a statment. Or i'm wrong here?

Since it's using JDatabaseQuery to build it, the OR won't be there on the
last loop if the where() function is called properly.

On Thursday, April 17, 2014, zero-24 notifications@github.com wrote:

@mbabker https://github.com/mbabker
ok. But least the last stament is wrong as it contains a "or" without a
statment. Or i'm wrong here?

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

hmm ok so why the manifest update fails?

Without being in front of a computer to test, couldn't tell you. Just
going off the top of my head at the moment.

On Thursday, April 17, 2014, zero-24 notifications@github.com wrote:

hmm ok so why the manifest update fails?

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

hmm ok @mbabker so i closing here as it not fix the issue correct.