session functions from session.inc, like session_save_session in D6, have been renamed to drupal_* equivalents, like drupal_save_session() in D7.

This should be added to the update guide at http://drupal.org/update/modules/6/7

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Project: Documentation » Drupal core
Version: » 7.x-dev
Component: Correction/Clarification » documentation

Moving to correct issue queue. We manage the update docs in the main Drupal issue queue. They don't get noticed here...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Issue tags: +Needs documentation updates

tagging

dmitrig01’s picture

dmitrig01 CreditAttribution: dmitrig01 commented
Status: Active » Fixed
dmitrig01’s picture

dmitrig01 CreditAttribution: dmitrig01 commented

http://drupal.org/update/modules/6/7#session_rename

yched’s picture

yched CreditAttribution: yched commented

The D6 and D7 code samples are the same :-)
D6 :
drupal_save_session(TRUE);

D7:
drupal_save_session(TRUE);

+ "All session functions starting with sess_*... " doesn't seem like a good regexp for session_save_session() ?

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Fixed » Needs work

Sounds like this needs a bit of work then?

dmitrig01’s picture

dmitrig01 CreditAttribution: dmitrig01 commented
Status: Needs work » Needs review

oops, sorry

dmitrig01’s picture

dmitrig01 CreditAttribution: dmitrig01 commented
Title: session_* functions renamed to drupal_*session* » sess_* functions renamed to drupal_*session*
jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Needs work

Looks like the code sample is fixed...

#5 also mentioned:

"All session functions starting with sess_*... " doesn't seem like a good regexp for session_save_session() ?

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs work » Fixed

I fixed http://drupal.org/update/modules/6/7#session_rename so that it has the correct d6 and d7 function names.

Status: Fixed » Closed (fixed)
Issue tags: -Needs documentation updates

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Needs documentation updates

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.