8000 extensionapi:info throws error · Issue #108 · TYPO3-coreapi/ext-coreapi · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Aug 15, 2023. It is now read-only.

extensionapi:info throws error #108

Closed
thomashohn opened this issue Sep 29, 2015 · 4 comments
Closed

extensionapi:info throws error #108

thomashohn opened this issue Sep 29, 2015 · 4 comments

Comments

@thomashohn
Copy link
Contributor

If ./cli_dispatch.phpsh extbase extensionapi:info is called with an extension that is not installed it throws the following error:
PHP Warning: Invalid argument supplied for foreach() in /Users/tho/flat-sysgit/ibog-extensions/coreapi/Classes/Command/ExtensionApiCommandController.php line 103
since the $data['em_conf'] is empty if a if (!empty($data['em_conf'] )) is placed around the foreach it will return that the extension is not installed instead of an run-time error
@etobi
Copy link
Member
etobi commented Sep 29, 2015

Hej,

thanks for your report and your suggestion. Could you please prepare and send a PullRequest? I will review and merge it then.

@thomashohn
Copy link
Contributor Author

Hi - if you could guide me to create the pull-request - I can do it :-)

@etobi
Copy link
Member
etobi commented Sep 29, 2015

Sure. You need to fork our repository into your github-account, checkout that repo, commit the bugfix, push it to github and then create a pull request.
Check https://help.github.com/articles/fork-a-repo/ and https://help.github.com/articles/using-pull-requests/ on how forking and PullRequests working on github.

Looking forward to your first PR :-)

regards
tobias

@thomashohn
Copy link
Contributor Author

Hi

Hope you have recieved a PR from me

Greetings

Thomas

2015-09-29 9:51 GMT+02:00 Tobias Liebig notifications@github.com:

Sure. You need to fork our repository into your github-account, checkout
that repo, commit the bugfix, push it to github and then create a pull
request.
Check https://help.github.com/articles/fork-a-repo/ and
https://help.github.com/articles/using-pull-requests/ on how forking and
PullRequests working on github.

Looking forward to your first PR :-)

regards
tobias

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

Venlig hilsen

Thomas Hohn
IT-udvikler
E-mail: tho@systime.dk
Tlf.: 30 36 08 16

http://systime.dk/
Adresse
https://www.google.dk/maps/place/Systime+A%2FS/@56.14616,10.201086,17z/data=!3m1!4b1!4m2!3m1!1s0x464c3f8bf58bade9:0xbbaee26f436fbfc3
| +45 7012 1100 | systime.dk

http://systimes.idebat.dk/

etobi added a commit that referenced this issue Sep 29, 2015
@etobi
Merge pull request #109 from thomashohn/Fix-non-installed-extension
5c90099 
[FIX] #108 #109 extensionapi:info throws error if extension does not exist
@etobi etobi closed this as completed Sep 29, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@etobi @thomashohn
0