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

Updated documentation go.php to 15 and adjusted copyright year. #14716

Merged
merged 1 commit into from Mar 18, 2019

Conversation

Projects
None yet
4 participants
@Faldon
Copy link
Contributor

Faldon commented Mar 15, 2019

Updated stable version

  • The stable version number now is 15
  • Updated copyright year to 2019

Fixes #14650

Updated documentation go.php to 15 and adjusted copyright year.
Signed-off-by: Thomas Pulzer <t.pulzer@thesecretgamer.de>

@Faldon Faldon force-pushed the Faldon:update_document_version branch from 8b20f30 to 7233c0e Mar 15, 2019

@kesselb

This comment has been minimized.

Copy link
Contributor

kesselb commented Mar 15, 2019

Fixes #14650

Only if someone uses "example" as theme 😎

@kesselb

This comment has been minimized.

Copy link
Contributor

kesselb commented Mar 15, 2019

#11304 we had some discussions about this (see the resolved conversation). I think we should go for \OC_Util::getVersion()[0] instead of 14,15,16...

@Faldon

This comment has been minimized.

Copy link
Contributor Author

Faldon commented Mar 16, 2019

Dynamic resolving would be definitely better. 😃
The reason why I thought "If any, it should go to the stable branch."

Of course, if the version number can be resolved dynamically, than that's the better approach. Is this just legacy remains or is there a reason that anything in the affected file is hardcoded respectively has no dependency on any other file?

@kesselb

This comment has been minimized.

Copy link
Contributor

kesselb commented Mar 16, 2019

Is this just legacy remains or is there a reason that anything in the affected file is hardcoded respectively has no dependency on any other file?

This file is not used anywhere. It's an example for a custom theme.

@Faldon

This comment has been minimized.

Copy link
Contributor Author

Faldon commented Mar 16, 2019

Ah, I see.
But to reproduce the issue #14650 I referenced, I set up a complete new development instance. If this file is for example themes, a fresh installation without any modification theme-wise should not call this code then, should it?
Edit:
Maybe in this case it's just the template of the LDAP plugin, that call's the wrong method to link to the documentation. I'll have another look into this.

@Faldon

This comment has been minimized.

Copy link
Contributor Author

Faldon commented Mar 18, 2019

It seems, the LDAP plugin does not use any curious method but the buildDocLinkToKey function in the legacy/defaults file.
After the copyright year in footer is already made dynamic in master branch and dynamically resolving the current version is indeed the much better approach, I will close this PR later that day.

@rullzer
Copy link
Member

rullzer left a comment

Fine for 15 IMO

@MorrisJobke
Copy link
Member

MorrisJobke left a comment

Fine with me 👍

@MorrisJobke MorrisJobke added this to the Nextcloud 15.0.6 milestone Mar 18, 2019

@MorrisJobke MorrisJobke merged commit da7579c into nextcloud:stable15 Mar 18, 2019

1 check passed

continuous-integration/drone/pr the build was successful
Details
@MorrisJobke

This comment has been minimized.

Copy link
Member

MorrisJobke commented Mar 18, 2019

/backport to master

@backportbot-nextcloud

This comment has been minimized.

Copy link

backportbot-nextcloud bot commented Mar 18, 2019

The backport to master failed. Please do this backport manually.

@Faldon Faldon deleted the Faldon:update_document_version branch Mar 25, 2019

@MorrisJobke

This comment has been minimized.

Copy link
Member

MorrisJobke commented Mar 26, 2019

Fixed on master already

@rullzer rullzer referenced this pull request Mar 29, 2019

Merged

15.0.6 RC 1 #14909

1 of 1 task complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.