Skip to content
This repository has been archived by the owner on Oct 4, 2017. It is now read-only.

Date #6

Merged
merged 3 commits into from Feb 14, 2012
Merged

Date #6

merged 3 commits into from Feb 14, 2012

Conversation

Fale
Copy link
Contributor

@Fale Fale commented Feb 14, 2012

Now it does work with qTranslate.
There is still the problem of future events, since I had to comment out the piece of code that was supposed to generate them since it was returning a Fatal Error. This should be fixed and re-implemented

@Fale Fale mentioned this pull request Feb 14, 2012
Merged
@@ -198,7 +198,7 @@ function get_date( $ID ) {
continue;

$date .= '<span class="' . $class . '" title="' . date( 'Y-m-d', strtotime( $value ) ) . '">';
$date .= $this->parent->api->apply_filters( 'date', $value, $field );
$date .= apply_filters( 'resume_date', $value, $field );
Copy link
Owner

Choose a reason for hiding this comment

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

These should in theory be equivalent.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree, they should... but the new one works, while the old one does not :(

Copy link
Owner

Choose a reason for hiding this comment

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

Just kidding. Found the mistake.

benbalter added a commit that referenced this pull request Feb 14, 2012
qTranslate error fixes; Props Fale.
@benbalter benbalter merged commit 999c26e into benbalter:develop Feb 14, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants