Skip to content

Test#1

Merged
ianmacl merged 1 commit intojoomla:masterfrom
ianmacl:master
Sep 28, 2011
Merged

Test#1
ianmacl merged 1 commit intojoomla:masterfrom
ianmacl:master

Conversation

@ianmacl
Copy link
Copy Markdown

@ianmacl ianmacl commented Sep 28, 2011

No description provided.

ianmacl pushed a commit that referenced this pull request Sep 28, 2011
@ianmacl ianmacl merged commit bd161af into joomla:master Sep 28, 2011
@eddieajau
Copy link
Copy Markdown
Contributor

I wouldn't have approved it anyway :)

ianmacl pushed a commit that referenced this pull request Sep 30, 2011
chdemko pushed a commit to chdemko/joomla-cms that referenced this pull request Nov 21, 2011
Add elseif declaration sniff
elkuku pushed a commit to elkuku/joomla-cms that referenced this pull request Jan 18, 2012
dextercowley referenced this pull request in dextercowley/joomla-cms Apr 11, 2013
@ghost ghost mentioned this pull request Jul 3, 2013
javigomez pushed a commit that referenced this pull request Oct 17, 2014
Add Fixes suggested by Sebastian Bergmann
robations added a commit to robations/joomla-cms that referenced this pull request Nov 20, 2014
If the expected manifest XML files/dirs are missing, the findManifest() method fails with:

    [ErrorException] array_merge(): Argument joomla#1 is not an array

because `$parentXmlfiles` and/or `$allXmlFiles` are false, rather than an array.

Caveat - although this fixed the problem for me, I don't fully understand the context of this code so perhaps there is a deeper problem? I was using the [Joomlatools Composer Installer](https://github.com/joomlatools/joomla-composer) to install a module and had deleted the contents of the tmp directory before this happened. Not sure why the tmp dir would be referenced to see if a module/component/plugin is installed, obviously the tmp dir will get cleared out periodically.
@exlemor exlemor mentioned this pull request Mar 8, 2025
4 tasks
@gwstyles gwstyles mentioned this pull request Nov 5, 2025
4 tasks
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