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

case sensitivity qcubed/Codegen/ #8

Open
matthiaz opened this issue Jan 16, 2018 · 8 comments
Open

case sensitivity qcubed/Codegen/ #8

matthiaz opened this issue Jan 16, 2018 · 8 comments

Comments

@matthiaz
Copy link

the appstarter creates a folder qcubed/Codegen/

that should be qcubed/CodeGen/

with a capital G, else the autoloader won't find it

@kukrik
Copy link
Member

kukrik commented Jan 16, 2018

Maybe I do not understand your question, but I'm working with QCubed 4v impeccably. In any case, I'll show you a small screenshot. Is there the same picture?

screen shot 2018-01-16 at 13 09 45

@matthiaz
Copy link
Author

matthiaz commented Jan 16, 2018 via email

@spekary
Copy link
Member

spekary commented Jan 16, 2018

If so, then yes, this is a problem. The issue is that the autoloader uses the name of the class to also be the name of the file, and some operating systems are case sensitive and some are not. In particular, the Mac OS is not, but Linux is, so those of us on Mac systems must be extra careful. The travis process should catch errors like this though.

@kukrik
Copy link
Member

kukrik commented Jan 16, 2018

To @matthiaz, I do not know how you got the application. Anyway I used the terminal command:

composer create-project --stability=dev qcubed/app-starter your_dir_name dev-qcubed4

That's only a question.

@spekary
Copy link
Member

spekary commented Jan 16, 2018

Everything that is "created" during the installation process is simply copied from the "install" directory in vendor/qcubed. This is a simple fix by changing the name of the file in there.

@matthiaz
Copy link
Author

@spekary I'm afraid I'm going to need some help with this.
I can't figure out where that Codegen or CodeGen folder is coming from.
I checked out the project qcubed/app-starter and qcubed/application but in the install folder I only find
Control
Jqui
Watcher

However, when running app-starter to create a new project those folders are present + the Codegen folder. Where is it coming from?

@spekary
Copy link
Member

spekary commented Jan 22, 2018

Well, travis is currently broken, and I am having a hard time fixing it. I think the problem is an API limit with github, but everything online suggests that there is no more API limit problem. But it sure looks like we are dealing with that.

Anyway, the folder you are asking about is created when installing the orm, which is required by the application. The folder itself is located at qcubed/orm/install/project/qcubed.

However, looking through the code, I believe the name of the folder is correct. The error is on line 3 of qcubed/orm/tools/codegen.php

use QCubed\Project\CodeGen\CodegenBase as Codegen;

should be

use QCubed\Project\Codegen\CodegenBase as Codegen;

@spekary
Copy link
Member

spekary commented Jan 24, 2018

I have fixed the travis problem, and fixed the datepicker problem.

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