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

Feature/import gc logs #720

Merged

Conversation

teiling88
Copy link
Member

1. Why is this change necessary?

2. What does this change do, exactly?

3. Describe each step to reproduce the issue or behaviour.

4. Please link to the relevant issues (if any).

5. Checklist

  • I have written tests and verified that they fail without my change
  • I have squashed any insignificant commits
  • This change has comments for package types, values, functions, and non-obvious lines of code

@codecov
Copy link

codecov bot commented Aug 10, 2018

Codecov Report

Merging #720 into development will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@              Coverage Diff               @@
##           development    #720      +/-   ##
==============================================
- Coverage         5.44%   5.43%   -0.01%     
==============================================
  Files              259     260       +1     
  Lines            16056   16073      +17     
==============================================
  Hits               874     874              
- Misses           15182   15199      +17
Impacted Files Coverage Δ
htdocs/src/Oc/Repository/AbstractEntity.php 100% <ø> (ø) ⬆️
...oCache/Persistence/GeoCache/GeoCacheRepository.php 8.97% <ø> (+0.05%) ⬆️
...Oc/Import/Logs/Controller/ImportLogsController.php 0% <0%> (ø)
htdocs/src/Oc/Command/CreateWebCacheCommand.php 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update acc00d4...00a4196. Read the comment docs.

@@ -25,7 +25,7 @@
"opencaching/okapi": "dev-master",
"opencaching/kitchensink-bundle": "1.0.6",
"endroid/qrcode": "^2.4",
"twbs/bootstrap": "^v4.0.0",
"twbs/bootstrap": "v4.0.0-beta.2",
Copy link
Member

Choose a reason for hiding this comment

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

why beta.2?

/** @var int */
public $geocacheId;

/** @var smallint */
Copy link
Member

Choose a reason for hiding this comment

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

small int should be int

/** @var int */
public $cacheId;

/** @var char */
Copy link
Member

Choose a reason for hiding this comment

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

char should be string

*/
public function isNew()
{
return $this->id=== null;
Copy link
Member

Choose a reason for hiding this comment

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

space missing before comparison characters

/** @var char */
public $language;

/** @var longtext */
Copy link
Member

Choose a reason for hiding this comment

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

longtext should be string

/** @var DateTime */
public $expires;

/** @var mediumblob */
Copy link
Member

Choose a reason for hiding this comment

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

mediumblob should be string

public $serviceName;

/** @var enum */
public $calltype;
Copy link
Member

Choose a reason for hiding this comment

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

should also be checked

/** @var string */
public $currentLogName;

/** @var bigint */
Copy link
Member

Choose a reason for hiding this comment

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

bigint should be int

/** @var string */
public $resourceName;

/** @var mediumint */
Copy link
Member

Choose a reason for hiding this comment

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

mediumint should be int

/** @var string */
public $password;

/** @var binary */
Copy link
Member

Choose a reason for hiding this comment

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

binary should be string

@teiling88 teiling88 merged commit 783378f into OpencachingDeutschland:development Aug 24, 2018
@teiling88 teiling88 deleted the feature/import-gc-logs branch December 7, 2018 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants