18F/federalist

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

Open

exports.down = function (db, callback) {
  createPassportTable(db).then(() => loadUsers(db)).then(users => migrateDownNextUser(users, db)).then(() => removeColumsFromUserTable(db))
    .then(() => {
      callback();
    })

• Read up
  Read up

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

Open

  deleteServiceInstance(name) {
    return this.fetchServiceInstance(name)
      .then(instance => this.accessToken().then(token => this.request(
        'DELETE',
        `/v2/service_instances/${instance.metadata.guid}?accepts_incomplete=true&recursive=true&async=true`,

• Read up
  Read up

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

Open

  deleteServiceInstance(name) {
    return this.fetchServiceInstance(name)
      .then(instance => this.accessToken().then(token => this.request(
        'DELETE',
        `/v2/service_instances/${instance.metadata.guid}?accepts_incomplete=true&recursive=true&async=true`,

• Read up
  Read up

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

Open

const migrateUpNextUser = (users, db) => {
  if (users.length > 0) {
    const user = users.pop();
    return migrateUpUserData(user, db).then(() => {
      console.log(`Migrated user: ${user.username}`);

• Read up
  Read up

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

Open

const migrateDownNextUser = (users, db) => {
  if (users.length > 0) {
    const user = users.pop();
    return insertPassportRowForUser(user, db).then(() => {
      console.log(`Migrated user: ${user.username}`);

• Read up
  Read up

Function exports has 86 lines of code (exceeds 25 allowed). Consider refactoring.

Open

module.exports = (sequelize, DataTypes) => {
  const Build = sequelize.define(
    'Build',
    {
      branch: {

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

Open

exports.down = (db, callback) => Promise.all([
    db.runSql('DROP INDEX IF EXISTS "build_site_idx";'),
    db.runSql('DROP INDEX IF EXISTS "build_user_idx";'),
    db.runSql('DROP INDEX IF EXISTS "site_users__user_sites_user_sites_idx";'),
    db.runSql('DROP INDEX IF EXISTS "site_users__user_sites_site_users_idx";'),

• Read up
  Read up

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

Open

exports.up = (db, callback) => Promise.all([
    db.runSql('CREATE INDEX IF NOT EXISTS "build_site_idx" ON build USING btree ("site");'),
    db.runSql('CREATE INDEX IF NOT EXISTS "build_user_idx" ON build USING btree ("user");'),
    db.runSql('CREATE INDEX IF NOT EXISTS "site_users__user_sites_user_sites_idx" ON site_users__user_sites USING btree ("user_sites");'),
    db.runSql('CREATE INDEX IF NOT EXISTS "site_users__user_sites_site_users_idx" ON site_users__user_sites USING btree ("site_users");'),

• Read up
  Read up

Function SiteContainer has 81 lines of code (exceeds 25 allowed). Consider refactoring.

Open

export function SiteContainer(props) {
  const { id } = useParams();
  const location = useLocation();
  const params = useSearchParams();
  const sites = useSelector(state => state.sites);

CloudFoundryAPIClient has 27 functions (exceeds 20 allowed). Consider refactoring.

Open

class CloudFoundryAPIClient {
  constructor({ apiUrl, authClient } = {}) {
    this.authClient = authClient ?? new CloudFoundryAuthClient();
    this.httpClient = new HttpClient(apiUrl ?? process.env.CLOUD_FOUNDRY_API_HOST);
  }

Function AddSite has 79 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function AddSite() {
  useEffect(() => () => {
    // dispatch the action to hide the additional new site fields
    // when this component is unmounted
    addNewSiteFieldsActions.hideAddNewSiteFields();

Function define has 77 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function define(sequelize, DataTypes) {
  const SiteBranchConfig = sequelize.define(
    'SiteBranchConfig',
    {
      branch: {

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

Open

    sender: {
      id: 693815, url: 'https://api.github.com/users/apburnes', type: 'User', login: 'apburnes', node_id: 'MDQ6VXNlcjY5MzgxNQ==', html_url: 'https://github.com/apburnes', gists_url: 'https://api.github.com/users/apburnes/gists{/gist_id}', repos_url: 'https://api.github.com/users/apburnes/repos', avatar_url: 'https://avatars.githubusercontent.com/u/693815?v=4', events_url: 'https://api.github.com/users/apburnes/events{/privacy}', site_admin: false, gravatar_id: '', starred_url: 'https://api.github.com/users/apburnes/starred{/owner}{/repo}', followers_url: 'https://api.github.com/users/apburnes/followers', following_url: 'https://api.github.com/users/apburnes/following{/other_user}', organizations_url: 'https://api.github.com/users/apburnes/orgs', subscriptions_url: 'https://api.github.com/users/apburnes/subscriptions', received_events_url: 'https://api.github.com/users/apburnes/received_events',
    },

• Read up
  Read up

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

Open

      user: {
        id: 32960241, url: 'https://api.github.com/users/maryclair', type: 'User', login: 'maryclair', node_id: 'MDQ6VXNlcjMyOTYwMjQx', html_url: 'https://github.com/maryclair', gists_url: 'https://api.github.com/users/maryclair/gists{/gist_id}', repos_url: 'https://api.github.com/users/maryclair/repos', avatar_url: 'https://avatars.githubusercontent.com/u/32960241?v=4', events_url: 'https://api.github.com/users/maryclair/events{/privacy}', site_admin: false, gravatar_id: '', starred_url: 'https://api.github.com/users/maryclair/starred{/owner}{/repo}', followers_url: 'https://api.github.com/users/maryclair/followers', following_url: 'https://api.github.com/users/maryclair/following{/other_user}', organizations_url: 'https://api.github.com/users/maryclair/orgs', subscriptions_url: 'https://api.github.com/users/maryclair/subscriptions', received_events_url: 'https://api.github.com/users/maryclair/received_events',
      },

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

  fetchServiceKey(name) {
    return this.fetchServiceKeys()
      .then(res => findEntity(res, name))
      .then(key => this.accessToken().then(token => this.request(
        'GET',

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

  createS3ServiceInstance(name, serviceName, spaceGuid, s3ServicePlanId) {
    return this.fetchS3ServicePlanGUID(serviceName, s3ServicePlanId)
      .then((servicePlanGuid) => {
        const body = {
          name,

• Read up
  Read up

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

Open

exports.up = (db, callback) => db.runSql(getSites)
    .then((sites) => Promise.all(sites.rows.map(site => db.runSql(cmdUpdateSiteSubdomain(site)))))
    .then(() => callback())
    .catch(callback);

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

  createS3ServiceInstance(name, serviceName, spaceGuid, s3ServicePlanId) {
    return this.fetchS3ServicePlanGUID(serviceName, s3ServicePlanId)
      .then((servicePlanGuid) => {
        const body = {
          name,

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

  fetchServiceKey(name) {
    return this.fetchServiceKeys()
      .then(res => findEntity(res, name))
      .then(key => this.accessToken().then(token => this.request(
        'GET',

• Read up
  Read up

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

Open

exports.down = (db, callback) => db.runSql(getSites)
    .then((sites) => Promise.all(sites.rows.map(site => db.runSql(cmdRevertSiteSubdomain(site)))))
    .then(() => callback())
    .catch(callback);

• Read up
  Read up