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

Weird error with new Topbar feature. #13

Closed
nurikabe opened this issue Nov 28, 2011 · 11 comments
Closed

Weird error with new Topbar feature. #13

nurikabe opened this issue Nov 28, 2011 · 11 comments

Comments

@nurikabe
Copy link

The autoloader expected class "Mopa\BootstrapBundle\Topbar\GenericTopbar" to be defined in file "/../vendor/bundles\Mopa\BootstrapBundle\Topbar\GenericTopbar.php". The file was found but the class was not in it, the class name or namespace probably has a typo.

@phiamo
Copy link
Owner

phiamo commented Nov 28, 2011

I changed the order of some parameter loading
Also had an issue on command line ...
could you try with actual version

@nurikabe
Copy link
Author

Sure, but what do you mean by "actual version"?

@phiamo
Copy link
Owner

phiamo commented Nov 28, 2011

how did you get the new version?
i think you should go

cd vendor/bundles/Mopa/BootstrapBundle
git pull

or try a vendors update

@nurikabe
Copy link
Author

Yes, that's what I did. This problem started after getting the most recent version.

@phiamo
Copy link
Owner

phiamo commented Nov 28, 2011

yeah and you should do it again, there was already another commit ...

@nurikabe
Copy link
Author

I have this version:

7fdcf63 Today 1:24 PM phiamo Fixed issue about scope on console

Seems to be the latest, but the problem persists.

@nurikabe
Copy link
Author

Duh. It's because TopBarInterface.php and GenericTopbar.php are using "<?" instead of "<?php"

Might want to stick to <?php if possible.

jonathaningram added a commit to jonathaningram/MopaBootstrapBundle that referenced this issue Nov 28, 2011
@chrisixion
Copy link

php short tag problem?

I changed

jonathaningram added a commit to jonathaningram/MopaBootstrapBundle that referenced this issue Nov 28, 2011
phiamo added a commit that referenced this issue Nov 28, 2011
@phiamo
Copy link
Owner

phiamo commented Nov 28, 2011

thnx for the info is fixed

@phiamo
Copy link
Owner

phiamo commented Nov 28, 2011

i also realized atm you MUST have KnpMenu installed ... i will work on this tonight

@phiamo
Copy link
Owner

phiamo commented Nov 28, 2011

sorry for inconveniance

@phiamo phiamo closed this as completed Nov 28, 2011
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

No branches or pull requests

3 participants