Skip to content

Updated it so project templates can use all keywords for folders - #114

Merged
elsassph merged 2 commits into
fdorg:developmentfrom
prankard:project-template-directory-args
Feb 6, 2014
Merged

elsassph merged 2 commits into
fdorg:developmentfrom
prankard:project-template-directory-args

Conversation

@prankard

Copy link
Copy Markdown
Contributor

It would be good to use this for project templates.
Some times I like to have extra folder imported into my directory like this:

  • src - package name - project name - project classes
  • src - package name - extensions - module 1 classes
  • src - package name - extensions - module 2 classes

I've tested and it works for almost all cases. Just doesn't work for the $(timestamp), however flashdevelop delivers you an error about the folder not being about to be created. However if you're messing around with variables in the project templates in folders, you should understand why that error would have occurred.

@elsassph

Copy link
Copy Markdown
Member

Nice, but why did you keep " if (dirName.ToUpper() == "$(PACKAGENAME)" || dirName.ToUpper() == "$(PACKAGEPATH)")" now that it is calling "dirName = ReplaceKeywords(dirName);"?

@prankard

Copy link
Copy Markdown
Contributor Author

Because I'm an idiot :P
I did this in an old repo. Tested it all fully worked in a lot of scenarios.
Re-downloaded the repo and put it back it without difference checking so that line got left in.

I've updated the file. Thanks for looking :)

Removed old directory name project template code
elsassph added a commit that referenced this pull request Feb 6, 2014
Updated it so project templates can use all keywords for folders
@elsassph
elsassph merged commit 173c6da into fdorg:development Feb 6, 2014
@prankard
prankard deleted the project-template-directory-args branch February 6, 2014 13:10
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