Skip to content

Fixes for TimestampFormatter - #7608

Merged
rullzer merged 1 commit into
masterfrom
pimp_timestampformatter
Dec 28, 2017
Merged

Fixes for TimestampFormatter#7608
rullzer merged 1 commit into
masterfrom
pimp_timestampformatter

Conversation

@rullzer

Copy link
Copy Markdown
Member

I was looking at the code (to use in the previewgenerator). And noticed some things that needed fixing.

Easy PR of the day.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzerrullzer added the 3. to review Waiting for reviews label Dec 22, 2017
@rullzerrullzer added this to the Nextcloud 13 milestone Dec 22, 2017
@rullzer
rullzerforce-pushed the pimp_timestampformatter branch from 624460f to 735b9fbCompareDecember 22, 2017 12:43
@codecov

codecovBot commented Dec 22, 2017

Copy link
Copy Markdown

Codecov Report

Merging #7608 into master will not change coverage.
The diff coverage is 0%.

@@ Coverage Diff @@## master #7608 +/- ##
=========================================
Coverage 51.17% 51.17% Complexity 24886 24886 =========================================
Files 1602 1602 Lines 94750 94750 Branches 1368 1368 =========================================
Hits 48485 48485 Misses 46265 46265
Impacted FilesCoverage ΔComplexity Δ
lib/private/Server.php81.43% <ø> (ø)134 <0> (ø)⬇️
core/register_command.php0% <0%> (ø)0 <0> (ø)⬇️
core/Command/Upgrade.php0% <0%> (ø)28 <0> (ø)⬇️
lib/private/Console/TimestampFormatter.php0% <0%> (ø)8 <0> (ø)⬇️
apps/files_trashbin/lib/Trashbin.php72.28% <0%> (-0.25%)136% <0%> (ø)
apps/files_trashbin/lib/Expiration.php91.93% <0%> (+1.61%)29% <0%> (ø)⬇️

@ChristophWurstChristophWurst 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.

Looks good

*/
public function hasStyle($name) {
$this->formatter->hasStyle($name);
return $this->formatter->hasStyle($name);

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.

oh 🙈

@ChristophWurstChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Dec 27, 2017
@rullzer
rullzer merged commit 46958e4 into masterDec 28, 2017
@rullzer
rullzer deleted the pimp_timestampformatter branch December 28, 2017 09:51
@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

4. to releaseReady to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@rullzer@ChristophWurst@blizzz