Skip to content

13.0.0 beta3 - #7475

Merged
MorrisJobke merged 1 commit into
masterfrom
13beta3
Dec 13, 2017
Merged

13.0.0 beta3#7475
MorrisJobke merged 1 commit into
masterfrom
13beta3

Conversation

@MorrisJobke

@MorrisJobkeMorrisJobke commented Dec 13, 2017

Copy link
Copy Markdown
Member

Includes:

@MorrisJobkeMorrisJobke added the 3. to review Waiting for reviews label Dec 13, 2017
@MorrisJobkeMorrisJobke added this to the Nextcloud 13 milestone Dec 13, 2017

@jospoortvlietjospoortvliet left a comment

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.

go for it...

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

I would block this until @blizzz had time to had a look into #7400 because we experience this as well.

@codecov

codecovBot commented Dec 13, 2017

Copy link
Copy Markdown

Codecov Report

Merging #7475 into master will increase coverage by 20.44%.
The diff coverage is 0%.

@@ Coverage Diff @@## master #7475 +/- ##
=============================================
+ Coverage 30.66% 51.11% +20.44% 
Complexity 24901 24901 =============================================
Files 1601 1601 Lines 94774 94774 Branches 1367 1367 =============================================
+ Hits 29059 48440 +19381 + Misses 65715 46334 -19381
Impacted FilesCoverage ΔComplexity Δ
version.php0% <0%> (ø)0 <0> (ø)⬇️
lib/private/Files/Storage/DAV.php65.4% <0%> (+0.21%)163% <0%> (ø)⬇️
lib/private/Files/Stream/Encryption.php75.22% <0%> (+0.45%)54% <0%> (ø)⬇️
apps/theming/lib/ThemingDefaults.php92.08% <0%> (+0.71%)48% <0%> (ø)⬇️
lib/private/Setup.php12.76% <0%> (+0.85%)50% <0%> (ø)⬇️
lib/public/AppFramework/Db/Mapper.php99.12% <0%> (+0.87%)30% <0%> (ø)⬇️
lib/private/legacy/image.php38.51% <0%> (+0.88%)208% <0%> (ø)⬇️
lib/private/Group/Group.php88.88% <0%> (+0.92%)52% <0%> (ø)⬇️
lib/private/App/DependencyAnalyzer.php95.59% <0%> (+1.25%)63% <0%> (ø)⬇️
apps/files_trashbin/lib/Expiration.php91.93% <0%> (+1.61%)29% <0%> (ø)⬇️
... and 375 more

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

I would block this until @blizzz had time to had a look into #7400 because we experience this as well.

Fix for this is in: #7479

@tiiiecherle

Copy link
Copy Markdown

It would be nice to see a fix for this #7380 included as this error is still present when using php 7.2 with the latest git nextcloud version.

According to
https://stackoverflow.com/questions/47656426/magento-2-2-1-unable-to-save-product-error-narrowing-occurred-during-type-infe

opcache_reset();

has to be included at the correct position.

Thanks in advance

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

opcache_reset();

We already do this:

opcache_reset();

That code is executed everytime the upgrade page is shown (just before you click the "Upgrade now" button.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

Rebased to latest master.

@tiiiecherle

Copy link
Copy Markdown

opcache_reset();
We already do this:

server/lib/base.php
Line 297 in c603f54
opcache_reset();
That code is executed everytime the upgrade page is shown (just before you click the "Upgrade now" button.

ok, but the error from #7380 is shown on every restart of the webserver. Would be nice to see this fixed as this is the last error in the log for me after upgrading to php 7.2 and using the latest git version of nextcloud from around 5 hours ago.

Thanks

@rullzer

Copy link
Copy Markdown
Member

@tiiiecherle we won't put an opcache_reset(); in the index.php. That would clear all caching of the compiled php files. This would put more load on your server and slow down each and all requests.

@MorrisJobke
MorrisJobke merged commit 66391d6 into masterDec 13, 2017
@MorrisJobke
MorrisJobke deleted the 13beta3 branch December 13, 2017 13:35
@tiiiecherle

Copy link
Copy Markdown

@tiiiecherle we won't put an opcache_reset(); in the index.php. That would clear all caching of the compiled php files. This would put more load on your server and slow down each and all requests.

ok, how can the error be solved? Or is it something that has to be fixed in the php packages?

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

ok, how can the error be solved? Or is it something that has to be fixed in the php packages?

It looks like this can only be fixed by an upcoming PHP release. I just checked and the opcache_reset does not solve the issue.

@tiiiecherle

Copy link
Copy Markdown

It looks like this can only be fixed by an upcoming PHP release. I just checked and the opcache_reset does not solve the issue.

ok, thanks. Perhaps the issue can be forwarded to their developers.

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

ok, thanks. Perhaps the issue can be forwarded to their developers.

AFAIK it is already fixed by PHP and the next release is in January.

@MorrisJobkeMorrisJobke mentioned this pull request Jan 2, 2018
30 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@MorrisJobke@tiiiecherle@rullzer@jospoortvliet