Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"#calendars" ≠ config_getstring(IMAPOPT_CALENDARPREFIX) #3382

Closed
dilyanpalauzov opened this issue Feb 24, 2021 · 0 comments
Closed

"#calendars" ≠ config_getstring(IMAPOPT_CALENDARPREFIX) #3382

dilyanpalauzov opened this issue Feb 24, 2021 · 0 comments

Comments

@dilyanpalauzov
Copy link
Contributor

Instead of using config_getstring(IMAPOPT_CALENDARPREFIX), jmap_calendar.c:get_schedule_address_set() and jmap_calendar.c:eventquery_search_run() use the constant "#calendars".

The usage in jmap_calendar.c:get_schedule_address_set() is tackled by #3381 .

  • Replace in jmap_calendar.c:eventquery_search_run() "#calendars" with config_getstring(IMAPOPT_CALENDARPREFIX)

  • Ideally load config_getstring(IMAPOPT_CALENDARPREFIX) once httpd is initialized in some variable and use that variable, instead of config_getstring(IMAPOPT_CALENDARPREFIX) across the code.
dilyanpalauzov added a commit to dilyanpalauzov/cyrus-imapd that referenced this issue Oct 22, 2021
@dilyanpalauzov
jmap_api.c: replace #calendars with config_getstring(IMAPOPT_CALENDAR…
089b82a 
…PREFIX)

Reported at cyrusimap#3382
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dilyanpalauzov