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.

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

Fix dev/joomla#1 - Fix list of events shown when adding Event Info and Registration pages #46

Merged
merged 1 commit into from
Jul 3, 2018

Conversation

aydun
Copy link
Contributor

@aydun aydun commented Jul 2, 2018

No description provided.

… page in Joomla

Also fixes similar issue when adding an Event Regisration page
'return.start_date' => 1,
'rowCount' => 100,
'return' => array("title"),
'start_date' => array('>=' => "today"),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be <=

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, we're looking for future events so where the start date is today or later.

'rowCount' => 100,
'is_online_registration' => 1,
'return' => array("title"),
'start_date' => array('>=' => "today"),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

@eileenmcnaughton
Copy link
Contributor

I'm going to merge this so it hits the rc - it seems there is a clearly agreed problem & you have worked on a fix - the fix is narrowly scoped & we are likely to struggle to get another Joomla! person to review - can you ask the reporter to test the 5.4 rc?

@eileenmcnaughton eileenmcnaughton merged commit 5cc8249 into civicrm:master Jul 3, 2018
@aydun
Copy link
Contributor Author

aydun commented Jul 4, 2018

Thanks for merging @eileenmcnaughton I have asked the reporter to test via SE

@aydun aydun changed the title Fix dev/joomla#1 - Fix list of events shown when adding an Event Info and Registration pages Fix dev/joomla#1 - Fix list of events shown when adding Event Info and Registration pages Jul 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants