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

Erweiterung wird als Abhängigkeit angezeigt, obwohl sie keine ist. #241

Open
garyee opened this issue Jan 26, 2015 · 8 comments
Open

Erweiterung wird als Abhängigkeit angezeigt, obwohl sie keine ist. #241

garyee opened this issue Jan 26, 2015 · 8 comments
Assignees
Labels

Comments

@garyee
Copy link

garyee commented Jan 26, 2015

unbenannt

Eine meiner Erweiterungen wird nur als Abhängigkeit angezeigt obwohl sie das nicht ist.
Sie besteht nicht aus so vielen Datein und die habe ich alle gecheckt. Der Abhängigkeitsbaum sieht aus wie bei allen anderen.
Der Fehler ist bis jetzt bei jeder Installation auf verschiedenen Servern und Contao Versionen aufgetreten.

https://github.com/garyee/contao-LinkContentElement
(Da ich die eigentlich nur für internen Gebrauch gebaut habe, installiere ich sie immer über gitHub ... deswegen auch die Readme Anweisungen. Wenn ich mal Zeit habe werde ich da mal was ordentliches schreiben)

Der ist an sich nicht so schlimm, aber es ist halt ziemlich nervig das man die Erweiterung nicht gleich sieht im Composer Client.

@garyee garyee changed the title Erweiterung wird als Abhängigkeit angezeigt, obwohl sie das nicht ist Erweiterung wird als Abhängigkeit angezeigt, obwohl sie keine ist. Jan 26, 2015
@garyee
Copy link
Author

garyee commented Jan 26, 2015

Und wenn ich auf den Namen der Erweiterung klicke dann kommt auch keine Detail-Seite sondern ein Fehler
unbenannt

@tristanlins
Copy link
Contributor

Das sieht so aus, als ob Composer die Erweiterung gar nicht als installiert erkennt?!
Die composer.lock (nicht die composer.json) aus dem Projekt wäre interessant um das Problem etwas genauer zu analysieren.

@tristanlins tristanlins added this to the 1.0 beta "bug fixing" milestone Jan 26, 2015
@garyee
Copy link
Author

garyee commented Jan 27, 2015

Hier die composer.lock

@discordier
Copy link
Member

Das ist seltsam. Sieht auf den ersten Blick gut aus.

@garyee
Copy link
Author

garyee commented Jan 27, 2015

Ihr könnt die Erweiterung sonst auch gern selbst installieren. Einfach dem GitHub-Link folgen und dann über die Anweisungen in der Readme das ganze über den Composer client installiert.

@discordier
Copy link
Member

Problem gefunden.
Eins vorne weg, das icon ist nicht das von "Dependency" sondern das von "removing". Er erkennt also dein Paket als zu deinstallieren.

Warum moechte er dies?
Ganz einfach, weil er es als "nicht zu installieren" erkennt.
Dies liegt zum einen daran, dass du dich nicht an die Konventionen gehalten hast und wir im composer-client nicht den Fall bedacht haben, dass dies jemand nicht tun wyrde, kleiner Fehler also auf beiden Seiten.

Die Konvention besagt:

While package names are case insensitive, the convention is all lowercase and dashes for word separation.

Du hingegen hast camel case verwendet, welches unser client nicht sauber erkennt. Wird behoben.

Abwenden kannst du dies, wenn du deinen package name der Konvention angleichst, also statt postyou/linkContentElement nimmst du postyou/link-content-element

@garyee
Copy link
Author

garyee commented Jan 27, 2015

AH ... manchmal sind es doch die "einfachsten" Sachen die Fehlerquelle. Die Konventionen habe ich schon gelesen, aber das ist mir doch glatt entfallen bzw. ist das wohl in der Flut der Informationen untergegangen, dazu noch ein Schuss Gewohnheit :) .
Vielen Dank fürs Fehler finden!

@discordier discordier removed this from the 1.0 beta "bug fixing" milestone Feb 26, 2016
@discordier
Copy link
Member

As you have moved the extension to https://github.com/postyou/contao-link-content-element, I guess we can savely close this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants