Skip to content

Clean up the SQL in markHavingUpdatedContent().#867

Merged
asmecher merged 1 commit intopkp:ojs-dev-2_4from
ubermichael:gh-1533
Jun 20, 2016
Merged

Clean up the SQL in markHavingUpdatedContent().#867
asmecher merged 1 commit intopkp:ojs-dev-2_4from
ubermichael:gh-1533

Conversation

@ubermichael
Copy link
Copy Markdown

  • The DISTINCT pdo.deposit_object_id is useless, since the query is
    grouped by the deposit_object_id column.
  • Make the last_modified column of the result aggregated to prevent
    database query errors.

Fixes pkp/pkp-lib#1533

 * The `DISTINCT pdo.deposit_object_id` is useless, since the query is
grouped by the deposit_object_id column.

 * Make the last_modified column of the result aggregated to prevent
database query errors.

Fixes pkp/pkp-lib#1533
@asmecher asmecher merged commit a0663f6 into pkp:ojs-dev-2_4 Jun 20, 2016
@asmecher
Copy link
Copy Markdown
Member

Likewise, I presume this needs picking into ojs-stable-2_4_8? (FYI, I tend to pick things forward without asking, but not backwards.)

@ubermichael
Copy link
Copy Markdown
Author

@asmecher Please do cherry-pick this one for the stable branch.

@asmecher
Copy link
Copy Markdown
Member

Cherry-picked!

@ubermichael ubermichael deleted the gh-1533 branch July 19, 2016 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants