Skip to content

[ticket/12509] Add an option to set the path of custom style directory (ie: email templates) for extensions#2455

Merged
nickvergessen merged 9 commits intophpbb:develop-ascraeusfrom
Nicofuma:ticket/12509
May 27, 2014
Merged

[ticket/12509] Add an option to set the path of custom style directory (ie: email templates) for extensions#2455
nickvergessen merged 9 commits intophpbb:develop-ascraeusfrom
Nicofuma:ticket/12509

Conversation

@Nicofuma
Copy link
Copy Markdown
Member

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ext_path

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If possible add the trailing slash here aswell.
Basically if it does not exist, set_custom_style() should append it

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or if it is required without, then the method should remove it

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's not really the name of the directory and i don't know what it is expected to be (ie: for the e-mails it's en_mail)
But it does not seem to be used anywhere else. So we should be able to change the thing to give directly the path to the directory.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in fact name should be the name of the style (ie: prosilver) the problem is that the method is used for two usages: define a style location and define a custom template directory.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing , here and in most of the other arrays

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the array from adm/index.php as a sample here

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

nickvergessen added a commit to nickvergessen/phpbb that referenced this pull request May 27, 2014
[ticket/12509] Add an option to set the path of custom style directory (ie: email templates) for extensions

* Nicofuma/ticket/12509:
  [ticket/12509] Add example in doc block
  [ticket/12509] Add missing ',' in arrays
  [ticket/12509] Update calls to set_custom_style in forgotten places
  [ticket/12509] Fix path in adm/index.php
  [ticket/12509] Fix typo in adm/index.php
  [ticket/12509] Fix doc block
  [ticket/12509] Fix the call to set_custom_file in adm/index and messenger
  [ticket/12509] Update acp custom_style setter
  [ticket/12509] Add an option to set the path of custom file for extensions
@nickvergessen nickvergessen merged commit 05cd25a into phpbb:develop-ascraeus May 27, 2014
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.

3 participants