Skip to content

Commit

Permalink
updated todo task
Browse files Browse the repository at this point in the history
  • Loading branch information
silverqx committed Aug 30, 2022
1 parent ecbb37d commit 724bab3
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions include/orm/tiny/relations/basepivot.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,10 @@ namespace Orm::Tiny::Relations
/*! Get the "related key" column name. */
const QString &getRelatedKey() const noexcept;

// TODO fuckup, timestamps in pivot, I will solve it when I will have to use timestamps in the code, anyway may be I will not need it, because I can pass to the method right away what I will need silverqx
// TODO fuckup, timestamps in pivot, the solution is to set CREATED_AT and UPDATED_AT right away in the fromAttributes method when I still have access to the parent, then I won't have to save a pointer to the parent. I can still save pointer to parent, but not for obtaining this timestamp column names. old - I will solve it when I will have to use timestamps in the code, anyway may be I will not need it, because I can pass to the method right away what I will need silverqx
// TODO also don't forget unsetRelations() if pivotParent will be implemented silverqx
/*! The parent model of the relationship. */
// template<typename Parent>
// inline static const Parent *pivotParent = nullptr;

protected:
Expand Down Expand Up @@ -125,7 +126,7 @@ namespace Orm::Tiny::Relations
/* We store off the parent instance so we will access the timestamp column names
for the model, since the pivot model timestamps aren't easily configurable
from the developer's point of view. We can use the parents to get these. */
// PivotModel::template pivotParent = &parent;
// pivotParent<Parent> = &parent;

instance.exists = exists;

Expand Down

0 comments on commit 724bab3

Please sign in to comment.