cerniglj1/osrs-quest-tool

View on GitHub

Showing 254 of 336 total issues

Similar blocks of code found in 7 locations. Consider refactoring.
Open

    "Rune Mysteries": {
        "miniquest": false,
        "shortName": "runeMysteries",
        "name": "Rune Mysteries",
        "url": "https://oldschool.runescape.wiki/w/Rune_Mysteries",
Severity: Major
Found in src/resources/questData.js and 6 other locations - About 2 hrs to fix
src/resources/questData.js on lines 530..548
src/resources/questData.js on lines 816..834
src/resources/questData.js on lines 1399..1417
src/resources/questData.js on lines 3055..3073
src/resources/questData.js on lines 3383..3401
src/resources/questData.js on lines 4162..4180

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 87.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 5 locations. Consider refactoring.
Open

  'Shield of Arrav': {
    miniquest: false,
    shortName: 'shieldOfArrav',
    name: 'Shield of Arrav',
    url: 'https://oldschool.runescape.wiki/w/Shield_of_Arrav',
Severity: Major
Found in lib/resources/quest/questData.js and 4 other locations - About 2 hrs to fix
lib/resources/quest/questData.js on lines 500..518
lib/resources/quest/questData.js on lines 2568..2586
lib/resources/quest/questData.js on lines 3047..3065
lib/resources/quest/questData.js on lines 3730..3748

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 87.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 5 locations. Consider refactoring.
Open

    "Merlin's Crystal": {
        "miniquest": false,
        "shortName": "merlinsCrystal",
        "name": "Merlin's Crystal",
        "url": "https://oldschool.runescape.wiki/w/Merlin%27s_Crystal",
Severity: Major
Found in src/resources/questData.js and 4 other locations - About 2 hrs to fix
src/resources/questData.js on lines 511..529
src/resources/questData.js on lines 2992..3010
src/resources/questData.js on lines 3658..3676
src/resources/questData.js on lines 3699..3717

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 87.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 2 locations. Consider refactoring.
Open

    requirements: {
      questPoints: 43,
      quests: [],
      skills: [
        {
Severity: Major
Found in lib/resources/quest/questData.js and 1 other location - About 1 hr to fix
lib/resources/quest/questData.js on lines 1344..1364

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 74.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 2 locations. Consider refactoring.
Open

    requirements: {
      questPoints: 20,
      quests: [],
      skills: [
        {
Severity: Major
Found in lib/resources/quest/questData.js and 1 other location - About 1 hr to fix
lib/resources/quest/questData.js on lines 4132..4152

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 74.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 2 locations. Consider refactoring.
Open

    requirements: {
      quests: ['In Search of the Myreque'],
      skills: [
        {
          skill: 'crafting',
Severity: Major
Found in lib/resources/quest/questData.js and 1 other location - About 1 hr to fix
lib/resources/quest/questData.js on lines 1467..1486

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 70.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 2 locations. Consider refactoring.
Open

    requirements: {
      quests: ['Fairytale I - Growing Pains'],
      skills: [
        {
          skill: 'thieving',
Severity: Major
Found in lib/resources/quest/questData.js and 1 other location - About 1 hr to fix
lib/resources/quest/questData.js on lines 2143..2162

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 70.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 9 locations. Consider refactoring.
Open

    rewards: {
      experience: [
        {
          skill: 'crafting',
          amount: 7000,
Severity: Major
Found in lib/resources/quest/questData.js and 8 other locations - About 1 hr to fix
lib/resources/quest/questData.js on lines 173..193
lib/resources/quest/questData.js on lines 1020..1040
lib/resources/quest/questData.js on lines 1365..1385
lib/resources/quest/questData.js on lines 2163..2183
lib/resources/quest/questData.js on lines 2516..2536
lib/resources/quest/questData.js on lines 2709..2729
lib/resources/quest/questData.js on lines 4070..4090
lib/resources/quest/questData.js on lines 4354..4374

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 65.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 9 locations. Consider refactoring.
Open

    rewards: {
      experience: [
        {
          skill: 'agility',
          amount: 10000,
Severity: Major
Found in lib/resources/quest/questData.js and 8 other locations - About 1 hr to fix
lib/resources/quest/questData.js on lines 173..193
lib/resources/quest/questData.js on lines 1020..1040
lib/resources/quest/questData.js on lines 1312..1332
lib/resources/quest/questData.js on lines 1365..1385
lib/resources/quest/questData.js on lines 2163..2183
lib/resources/quest/questData.js on lines 2709..2729
lib/resources/quest/questData.js on lines 4070..4090
lib/resources/quest/questData.js on lines 4354..4374

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 65.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 9 locations. Consider refactoring.
Open

    rewards: {
      experience: [
        {
          skill: 'attack',
          amount: 2000,
Severity: Major
Found in lib/resources/quest/questData.js and 8 other locations - About 1 hr to fix
lib/resources/quest/questData.js on lines 173..193
lib/resources/quest/questData.js on lines 1020..1040
lib/resources/quest/questData.js on lines 1312..1332
lib/resources/quest/questData.js on lines 1365..1385
lib/resources/quest/questData.js on lines 2516..2536
lib/resources/quest/questData.js on lines 2709..2729
lib/resources/quest/questData.js on lines 4070..4090
lib/resources/quest/questData.js on lines 4354..4374

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 65.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 9 locations. Consider refactoring.
Open

    rewards: {
      experience: [
        {
          skill: 'construction',
          amount: 250,
Severity: Major
Found in lib/resources/quest/questData.js and 8 other locations - About 1 hr to fix
lib/resources/quest/questData.js on lines 173..193
lib/resources/quest/questData.js on lines 1020..1040
lib/resources/quest/questData.js on lines 1312..1332
lib/resources/quest/questData.js on lines 1365..1385
lib/resources/quest/questData.js on lines 2163..2183
lib/resources/quest/questData.js on lines 2516..2536
lib/resources/quest/questData.js on lines 2709..2729
lib/resources/quest/questData.js on lines 4070..4090

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 65.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 9 locations. Consider refactoring.
Open

    rewards: {
      experience: [
        {
          skill: 'agility',
          amount: 20000,
Severity: Major
Found in lib/resources/quest/questData.js and 8 other locations - About 1 hr to fix
lib/resources/quest/questData.js on lines 173..193
lib/resources/quest/questData.js on lines 1020..1040
lib/resources/quest/questData.js on lines 1312..1332
lib/resources/quest/questData.js on lines 1365..1385
lib/resources/quest/questData.js on lines 2163..2183
lib/resources/quest/questData.js on lines 2516..2536
lib/resources/quest/questData.js on lines 4070..4090
lib/resources/quest/questData.js on lines 4354..4374

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 65.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 9 locations. Consider refactoring.
Open

    rewards: {
      experience: [
        {
          skill: 'attack',
          amount: 2500,
Severity: Major
Found in lib/resources/quest/questData.js and 8 other locations - About 1 hr to fix
lib/resources/quest/questData.js on lines 173..193
lib/resources/quest/questData.js on lines 1020..1040
lib/resources/quest/questData.js on lines 1312..1332
lib/resources/quest/questData.js on lines 1365..1385
lib/resources/quest/questData.js on lines 2163..2183
lib/resources/quest/questData.js on lines 2516..2536
lib/resources/quest/questData.js on lines 2709..2729
lib/resources/quest/questData.js on lines 4354..4374

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 65.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 9 locations. Consider refactoring.
Open

    rewards: {
      experience: [
        {
          skill: 'crafting',
          amount: 1000,
Severity: Major
Found in lib/resources/quest/questData.js and 8 other locations - About 1 hr to fix
lib/resources/quest/questData.js on lines 1020..1040
lib/resources/quest/questData.js on lines 1312..1332
lib/resources/quest/questData.js on lines 1365..1385
lib/resources/quest/questData.js on lines 2163..2183
lib/resources/quest/questData.js on lines 2516..2536
lib/resources/quest/questData.js on lines 2709..2729
lib/resources/quest/questData.js on lines 4070..4090
lib/resources/quest/questData.js on lines 4354..4374

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 65.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 9 locations. Consider refactoring.
Open

    rewards: {
      experience: [
        {
          skill: 'agility',
          amount: 15000,
Severity: Major
Found in lib/resources/quest/questData.js and 8 other locations - About 1 hr to fix
lib/resources/quest/questData.js on lines 173..193
lib/resources/quest/questData.js on lines 1312..1332
lib/resources/quest/questData.js on lines 1365..1385
lib/resources/quest/questData.js on lines 2163..2183
lib/resources/quest/questData.js on lines 2516..2536
lib/resources/quest/questData.js on lines 2709..2729
lib/resources/quest/questData.js on lines 4070..4090
lib/resources/quest/questData.js on lines 4354..4374

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 65.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 9 locations. Consider refactoring.
Open

    rewards: {
      experience: [
        {
          skill: 'crafting',
          amount: 2000,
Severity: Major
Found in lib/resources/quest/questData.js and 8 other locations - About 1 hr to fix
lib/resources/quest/questData.js on lines 173..193
lib/resources/quest/questData.js on lines 1020..1040
lib/resources/quest/questData.js on lines 1312..1332
lib/resources/quest/questData.js on lines 2163..2183
lib/resources/quest/questData.js on lines 2516..2536
lib/resources/quest/questData.js on lines 2709..2729
lib/resources/quest/questData.js on lines 4070..4090
lib/resources/quest/questData.js on lines 4354..4374

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 65.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 3 locations. Consider refactoring.
Open

class SlayerUnlock {
  constructor({ name, cost, description }) {
    this.name = name;
    this.cost = cost;
    this.description = description;
Severity: Major
Found in lib/model/slayer/SlayerReward.js and 2 other locations - About 1 hr to fix
lib/model/slayer/SlayerReward.js on lines 8..14
lib/model/slayer/SlayerReward.js on lines 15..21

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 64.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 3 locations. Consider refactoring.
Open

class SlayerBuy {
  constructor({ item, priceSoldAt, notes }) {
    this.item = item;
    this.priceSoldAt = priceSoldAt;
    this.notes = notes;
Severity: Major
Found in lib/model/slayer/SlayerReward.js and 2 other locations - About 1 hr to fix
lib/model/slayer/SlayerReward.js on lines 1..7
lib/model/slayer/SlayerReward.js on lines 8..14

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 64.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 3 locations. Consider refactoring.
Open

class SlayerExtend {
  constructor({ name, cost, notes }) {
    this.name = name;
    this.cost = cost;
    this.notes = notes;
Severity: Major
Found in lib/model/slayer/SlayerReward.js and 2 other locations - About 1 hr to fix
lib/model/slayer/SlayerReward.js on lines 1..7
lib/model/slayer/SlayerReward.js on lines 15..21

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 64.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Similar blocks of code found in 3 locations. Consider refactoring.
Open

    requirements: {
      quests: ['Druidic Ritual', 'Troll Stronghold'],
      skills: [
        {
          skill: 'herblore',
Severity: Major
Found in lib/resources/quest/questData.js and 2 other locations - About 55 mins to fix
lib/resources/quest/questData.js on lines 652..666
lib/resources/quest/questData.js on lines 1609..1623

Duplicated Code

Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

Tuning

This issue has a mass of 54.

We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

Refactorings

Further Reading

Severity
Category
Status
Source
Language