Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
I've installed the module on Oracle and it crashed on messages listing and threat view. Here's a patch.
Comment | File | Size | Author |
---|---|---|---|
#7 | 1840428-oracle-support.patch | 8.63 KB | metallized |
privatemsg.module.patch | 3.16 KB | Petrica | |
Comments
Comment #1
BerdirNo need to keep the commented out line here, just remove it.
Also, I think we should actually fix privatemsg_user_load_multiple() to not blow up when called with uid 0. Which ultimately is a core bug that is known for a long time now :(
Woah, black oracle magic :)
if that works for you then that sounds fine to me. Two remarks:
- You should use :current_uid or something like that, db placeholders shouldn't be in upper case.
- There are some tabs here, that also unecessarly change the mysql query below.
Comment #2
metallized CreditAttribution: metallized commentedHi, i got the 7.x.1.3 version of the module and apply this patch but i cannot get into the messages page i'm getting this error:
PDOException: SELECT * FROM (SELECT TAB.*, ROWNUM RWN_TO_REMOVE FROM (SELECT pmi.thread_id AS thread_id, MIN(pm.subject) AS subject, MAX(pm.timestamp) AS last_updated, MAX(pm.has_tokens) AS has_tokens, SUM(pmi.is_new) AS is_new, (SELECT distinct LISTAGG(pmia.type || '_' || pmia.recipient,',') WITHIN GROUP (ORDER BY pmia.recipient) FROM userremote.pm_index pmia WHERE pmia.type <> 'hidden' AND pmia.thread_id = pmi.thread_id AND pmia.recipient <> :currentUID) AS participants FROM {pm_message} pm INNER JOIN {pm_index} pmi ON pm.mid = pmi.mid WHERE (pmi.recipient = :db_condition_placeholder_0) AND (pmi.type IN (:db_condition_placeholder_1, :db_condition_placeholder_2)) AND (pmi.deleted = :db_condition_placeholder_3) GROUP BY pmi.thread_id ORDER BY last_updated DESC) TAB) WHERE RWN_TO_REMOVE BETWEEN 1 AND 25 (prepared: SELECT * FROM (SELECT TAB.*, ROWNUM RWN_TO_REMOVE FROM (SELECT pmi.thread_id AS thread_id, MIN(pm.subject) AS subject, MAX(pm.timestamp) AS last_updated, MAX(pm.has_tokens) AS has_tokens, SUM(pmi.is_new) AS is_new, (SELECT distinct LISTAGG(pmia.type || '_' || pmia.recipient,',') WITHIN GROUP (ORDER BY pmia.recipient) FROM userremote.pm_index pmia WHERE pmia.type <> 'hidden' AND pmia.thread_id = pmi.thread_id AND pmia.recipient <> :currentUID) AS participants FROM "PM_MESSAGE" pm INNER JOIN "PM_INDEX" pmi ON pm.mid = pmi.mid WHERE (pmi.recipient = :db_condition_placeholder_0) AND (pmi.type IN (:db_condition_placeholder_1, :db_condition_placeholder_2)) AND (pmi.deleted = :db_condition_placeholder_3) GROUP BY pmi.thread_id ORDER BY last_updated DESC) TAB) WHERE RWN_TO_REMOVE BETWEEN 1 AND 25 ) e: SQLSTATE[HY000]: General error: 942 OCIStmtExecute: ORA-00942: table or view does not exist (/root/PDO_OCI-1.0/oci_statement.c:142) args: Array ( [:db_condition_placeholder_0] => 1 [:db_condition_placeholder_1] => hidden [:db_condition_placeholder_2] => user [:db_condition_placeholder_3] => 0 [:currentUID] => 1 ) en PagerDefault->execute() (línea 79 de /var/www/html/includes/pager.inc).
Some advice is highly appreciated. Suscribe.
Comment #3
metallized CreditAttribution: metallized commentedHi, i was able to get work this module over the 7.x-1.4 version. here is the diff i got can anyone help me making this a patch?
Comment #4
metallized CreditAttribution: metallized commentedComment #6
metallized CreditAttribution: metallized commentedSorry wrong patch...
Comment #7
metallized CreditAttribution: metallized commentedHere is
Comment #8
metallized CreditAttribution: metallized commentedComment #9
metallized CreditAttribution: metallized commentedComment #10
metallized CreditAttribution: metallized commentedComment #12
oadaeh CreditAttribution: oadaeh as a volunteer commentedSince this is stated on the project's home page:
"If you are interested in a stable and tested version of Privatemsg, use the latest stable 1.x release. No new features will be added to these branches and the API and UI is stable and will not be changed (including strings for translation)."
This patch will not go into the 1.x branch.
If you want to update it for the 2.x branch, maybe it will go in there.
Comment #13
soraver CreditAttribution: soraver commentedPatch in #3 worked for me too.
Comment #14
ivnish CreditAttribution: ivnish commentedComment #15
andypostD7 is not yet outdated