Follow up for #1498674: Refactor node properties to multilingual →
Simplify code
In 1498674 was the @todo
// @todo Introduce node_mass_delete() or make node_mass_update() more flexible.
On the line in the patch in comment #303:
2850
The line of the final patch might change but that info might help to find them.
For example,
function node_user_predelete($account) {
// Delete nodes (current revisions).
// @todo Introduce node_mass_delete() or make node_mass_update() more flexible.
- $nodes = db_select('node', 'n')
+ $nodes = db_select('node_field_data', 'n')
+ ->distinct()
->fields('n', array('nid'))
->condition('uid', $account->uid)
->execute()
->fetchCol();
node_delete_multiple($nodes);
// Delete old revisions.
- $revisions = db_query('SELECT vid FROM {node_revision} WHERE uid = :uid', array(':uid' => $account->uid))->fetchCol();
+ $revisions = db_query('SELECT DISTINCT vid FROM {node_field_revision} WHERE uid = :uid', array(':uid' => $account->uid))->fetchCol();
foreach ($revisions as $revision) {
node_revision_delete($revision);
}
Introduce node_mass_delete() or make node_mass_update() more flexible.
No.
maybe
Active
11.0 🔥
node system
(Drupal 8 Multilingual Initiative) is the tag used by the multilingual initiative to mark core issues (and some contributed module issues). For versions other than Drupal 8, use the i18n (Internationalization) tag on issues which involve or affect multilingual / multinational support. That is preferred over Translation.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
This code and comment still exists almost 12 years later XD
I think this is still relevant.
Let's tie this to the new NodeRepository service that
📌
Create a new NodeRepository for the code from Drupal\node\NodeStorage
Active
will introduce, given that the code this issue would replace directly calls functions from that service (userRevisionIds)
I also tracked this @todo down originally to #8: Let users cancel their accounts →